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

bpo-43737: More useful documentation on behavior of % when divisor is negative #25220

Closed
wants to merge 1 commit into from

Conversation

TonyFlury
Copy link
Contributor

@TonyFlury TonyFlury commented Apr 6, 2021

@terryjreedy
Copy link
Member

Conflict is trivial and easy to fix. I did not do it because A. Tony needs to learn, and B. PR has other changes, not related to issues, and needs to be cleaned up or redone. I expect the latter will be needed.

@TonyFlury
Copy link
Contributor Author

Withdrawing pull request - will try again once I understand what I did wrong ....

@TonyFlury TonyFlury closed this Apr 8, 2021
@TonyFlury TonyFlury deleted the issue-43737 branch April 8, 2021 00:54
@terryjreedy
Copy link
Member

Before you delete the issue-43737 branch, you can check it out and open the files with the changes you wrote. The editor should keep the copy even when you checkout master (and update it!) and delete this branch. If the branch version of the file has both good and bad changes, open the master version in a 2nd editor and copy and paste the good changes into that. (If you are not using IDLE otherwise, it will serve as editor 2.)

After pushing a revised or new branch, and pushing the black on white [ New pull request] on github, but before pushing the green button that 'publishes' the proposed PR, look at all the files listed and the diff for each to make sure the PR has all and only the changes intended and written by you for the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review docs Documentation in the Doc dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants