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

gh-85046: Add EHWPOISON error code #126397

Closed
wants to merge 1 commit into from
Closed

Conversation

rruuaanng
Copy link
Contributor

@rruuaanng rruuaanng commented Nov 4, 2024


📚 Documentation preview 📚: https://cpython-previews--126397.org.readthedocs.build/

https://man7.org/linux/man-pages/man3/errno.3.html
The errno module is quite complete, but it lacks this EHWPOISON.

@rruuaanng
Copy link
Contributor Author

And some error codes are missing in the document. I'll finish them tomorrow.

@ZeroIntensity
Copy link
Member

@rruuaanng Please don't edit out the Issue: section that the bot adds.

@rruuaanng
Copy link
Contributor Author

Sorry, I didn't seem to pay attention.

@erlend-aasland
Copy link
Contributor

Sorry, I didn't seem to pay attention.

You've been told multiple times to read the devguide; please pay attention and do so. It is clear from your pattern of behaviour that you do not have a clear concept of our development workflow. Please focus on the first section, Getting Started.

https://devguide.python.org

@erlend-aasland
Copy link
Contributor

And some error codes are missing in the document. I'll finish them tomorrow.

This is why you should test your changes locally before creating a PR. Please read the devguide.

@rruuaanng
Copy link
Contributor Author

And some error codes are missing in the document. I'll finish them tomorrow.

This is why you should test your changes locally before creating a PR. Please read the devguide.

It refers to my other PR.

@rruuaanng

This comment was marked as duplicate.

@erlend-aasland
Copy link
Contributor

erlend-aasland commented Nov 6, 2024

Please add this change to #126420.

@rruuaanng
Copy link
Contributor Author

Please add this change to #126420.

No problem. I think it might be better this way.

@erlend-aasland
Copy link
Contributor

Please add this change to #126420.

Ah, this actually adds a new constant. Then it does not belong to the documentation issue you've linked this to. Please see if there is an open issue; if not, create an issue for this.

@rruuaanng
Copy link
Contributor Author

Please add this change to #126420.

Ah, this actually adds a new constant. Then it does not belong to the documentation issue you've linked this to. Please see if there is an open issue; if not, create an issue for this.

It's issue is what the PR title point to. If you don't mind, you can help me reopen this PR first and then let me do other work :)

@erlend-aasland
Copy link
Contributor

No, it is not. The issue is about missing documentation, not missing code. There is a big difference! Missing documentation would be a bug in the documentation, so any linked PRs would be backported to all supported branches. On the other side, missing error codes are new features; they cannot be backported! Please understand this. This is explained in the devguide; please read it!

Please do not reopen this PR; focus only on your existing PRs before opening any new PR at all.

@rruuaanng
Copy link
Contributor Author

Oh, I apologize. My thought is that new content should be reviewed separately from documentation changes. Thank you for your attention to this!

@erlend-aasland
Copy link
Contributor

My thought is that new content should be reviewed separately from documentation changes. Thank you for your attention to this!

That is a correct assumption, but they are (mostly) also separated by issue. The general idea is that you should think of issues like you would think of Good Commits (devguide-link; please read it!): don't mix features and bug-fixes in the same issue (and definitely not in a commit).

Again, I urge you to focus only on your currently open PRs. Try not to lose focus by opening new PRs/issues nor by reviewing other work. I think you will learn more and be more successful by being more focused (and I also think this will greatly reduce reviewer friction). Please consider my advice.

@rruuaanng
Copy link
Contributor Author

Erlend, thank you so much. It seems like most of my PRs have been reviewed by you. I know you've mentioned these points many times, and I apologize for that. I really have been reading the development guidelines, but sometimes I might not notice when I violate them, which stems from my mindset being stuck in another development mode (in fact, I didn't even know what CI was at first). If I violate the development rules again while working on other PRs, I hope you can remind me! In the future, I will focus on completing the PRs I open, and although I may spend some time helping others or reviewing, it won't detract from my commitment to my own PRs. Thank you for taking the time to guide me. I really appreciate it!

@erlend-aasland
Copy link
Contributor

I really have been reading the development guidelines, but sometimes I might not notice when I violate them, which stems from my mindset being stuck in another development mode [...]

This is why I'm suggesting that you focus on your open PRs. You will learn better by focusing your efforts. Remember also that landing a PR can be time-consuming. It may take several months in some cases! Also remember that it is ok for PRs to be rejected; that happens all the time, so do not take it personally.

[...] although I may spend some time helping others or reviewing, it won't detract from my commitment to my own PRs [...]

I will advice you again from doing this; after resolving your open PRs, you can try again to perform a couple of reviews, but I would rather not encourage you to learn these two development workflows at the same time. Please consider my advice.

If I violate the development rules again while working on other PRs, I hope you can remind me!

It is ok to forget things, especially for early contributors, but the repeated patterns of not following our development workflow indicates that more time needs to be spent with the devguide before continuing your contributions. This is perfectly fine. I do not expect you to know the entire devguide by heart (none of us do!), but I do expect you to read the sections I link to and learn from your failures. I know I'm very direct with you, but it is with the best intentions.

@rruuaanng rruuaanng changed the title gh-85046: Add EHWPOISON error code gh-126585: Add EHWPOISON error code Nov 8, 2024
@rruuaanng
Copy link
Contributor Author

@erlend-aasland here :)

@rruuaanng
Copy link
Contributor Author

I should probably open a new PR.

@rruuaanng rruuaanng deleted the gh85046 branch November 8, 2024 15:04
@rruuaanng rruuaanng changed the title gh-126585: Add EHWPOISON error code gh-85046: Add EHWPOISON error code Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants