X Tutup
The Wayback Machine - https://web.archive.org/web/20221223110328/https://github.com/python/cpython/pull/24171
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bpo-42269: Add slots parameter to dataclass decorator #24171

Merged
merged 8 commits into from May 1, 2021
Merged

bpo-42269: Add slots parameter to dataclass decorator #24171

merged 8 commits into from May 1, 2021

Conversation

uriyyo
Copy link
Member

@uriyyo uriyyo commented Jan 8, 2021 •
edited by bedevere-bot

@uriyyo uriyyo requested a review from ericvsmith as a code owner Jan 8, 2021
@uriyyo uriyyo changed the title bpo-42269: Add slots parameter to dataclass decorator bpo-42269: Add slots parameter to dataclass decorator Jan 22, 2021
@github-actions
Copy link

github-actions bot commented Feb 22, 2021

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions bot added the stale Stale PR or inactive for long period of time. label Feb 22, 2021
@uriyyo
Copy link
Member Author

uriyyo commented Feb 25, 2021

Hi @ericvsmith, could you please review this PR?)

@github-actions github-actions bot removed the stale Stale PR or inactive for long period of time. label Feb 26, 2021
ericvsmith
ericvsmith requested changes Apr 28, 2021
View changes
Copy link
Member

@ericvsmith ericvsmith left a comment

Thanks for fixing the conflicts, @uriyyo!

Doc/library/dataclasses.rst Outdated Show resolved Hide resolved
Lib/test/test_dataclasses.py Show resolved Hide resolved
@bedevere-bot
Copy link

bedevere-bot commented Apr 28, 2021

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@uriyyo
Copy link
Member Author

uriyyo commented Apr 28, 2021

I have made the requested changes; please review again

@bedevere-bot
Copy link

bedevere-bot commented Apr 28, 2021

Thanks for making the requested changes!

@ericvsmith: please review the changes made to this pull request.

@bedevere-bot bedevere-bot requested a review from ericvsmith Apr 28, 2021
@uriyyo
Copy link
Member Author

uriyyo commented Apr 28, 2021

@ericvsmith Could you please verify updated docs?

Should we mention this changes at python 3.10 whats new section?

ericvsmith
ericvsmith requested changes Apr 28, 2021
View changes
Copy link
Member

@ericvsmith ericvsmith left a comment

I have a few minor grammar nits, otherwise this looks good.

A whats new section for dataclasses is probably a good idea. I also need to add kw_only and match_args, too. I'll add kw_only and match_args after this PR lands.

I hope to get this all committed this weekend so it makes beta1.

Doc/library/dataclasses.rst Outdated Show resolved Hide resolved
Doc/library/dataclasses.rst Outdated Show resolved Hide resolved
@bedevere-bot
Copy link

bedevere-bot commented Apr 28, 2021

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@uriyyo
Copy link
Member Author

uriyyo commented Apr 28, 2021

I have made the requested changes; please review again

@bedevere-bot
Copy link

bedevere-bot commented Apr 28, 2021

Thanks for making the requested changes!

@ericvsmith: please review the changes made to this pull request.

@bedevere-bot bedevere-bot requested a review from ericvsmith Apr 28, 2021
@uriyyo
Copy link
Member Author

uriyyo commented Apr 28, 2021

Regarding the Whats new. Will you mention the slots feature or should I do it?

@ericvsmith
Copy link
Member

ericvsmith commented Apr 28, 2021

You should add slots, I’ll add the others in a different PR.

@uriyyo
Copy link
Member Author

uriyyo commented Apr 28, 2021

@ericvsmith Could you please review whats new?

@ericvsmith
Copy link
Member

ericvsmith commented Apr 28, 2021

Thanks, @uriyyo. This all looks good. I'll take one last pass at reviewing it this weekend and commit it.

@ericvsmith ericvsmith merged commit c241991 into python:master May 1, 2021
12 checks passed
kreathon pushed a commit to kreathon/cpython that referenced this pull request May 2, 2021
@uriyyo @kreathon
bpo-42269: Add slots parameter to dataclass decorator (pythonGH-24171)
a53792a 
Add slots parameter to dataclass decorator and make_dataclass function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericvsmith ericvsmith

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@uriyyo @bedevere-bot @ericvsmith @the-knights-who-say-ni
X Tutup