Skip to content

[3.10] bpo-46120: State that | is preferred over Union (GH-30222) #30250

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

Merged
merged 1 commit into from
Dec 29, 2021

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 24, 2021

Co-authored-by: Éric [email protected]
Co-authored-by: Guido van Rossum [email protected]
(cherry picked from commit 1b30660)

Co-authored-by: Nikita Sobolev [email protected]

https://bugs.python.org/issue46120

Automerge-Triggered-By: GH:gpshead

@miss-islington
Copy link
Contributor Author

@sobolevn and @gpshead: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@sobolevn and @gpshead: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@sobolevn and @gpshead: Status check is done, and it's a success ❌ .

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ❌ .

@ambv
Copy link
Contributor

ambv commented Dec 28, 2021

This is annoying. Without my last commit the build was failing with:

WARNING: [whatsnew/changelog:86] "::" found in "Lib/ctypes/test/test_functions.py::test_mro"
make[1]: *** [Makefile:51: build] Error 1
make[1]: Leaving directory '/home/runner/work/cpython/cpython/Doc'
Suspicious check complete; look for any errors in the above output or in build/suspicious/suspicious.csv.  If all issues are false positives, append that file to tools/susp-ignored.csv.
make: *** [Makefile:115: suspicious] Error 1

With the commit adding the ignore, it's failing with:

WARNING: Found 1/384 unused rules: whatsnew/changelog,,::,Lib/ctypes/test/test_functions.py::test_mro
make[1]: *** [Makefile:51: build] Error 1
make[1]: Leaving directory '/home/runner/work/cpython/cpython/Doc'
Suspicious check complete; look for any errors in the above output or in build/suspicious/suspicious.csv.  If all issues are false positives, append that file to tools/susp-ignored.csv.
make: *** [Makefile:115: suspicious] Error 1

Needless to say, the second error doesn't reproduce locally for me.

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

2 similar comments
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@sobolevn
Copy link
Member

@ambv sorry, looks like this is my fault with a bad NEWS entry 🙂

See #30259

Co-authored-by: Éric <[email protected]>
Co-authored-by: Guido van Rossum <[email protected]>
(cherry picked from commit 1b30660)

Co-authored-by: Nikita Sobolev <[email protected]>
@ambv ambv force-pushed the backport-1b30660-3.10 branch from b2f3eb0 to 2c6a053 Compare December 29, 2021 11:21
@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

2 similar comments
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Base branch was modified. Review and try the merge again..

@miss-islington
Copy link
Contributor Author

@miss-islington: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit ed1671c into python:3.10 Dec 29, 2021
@miss-islington miss-islington deleted the backport-1b30660-3.10 branch December 29, 2021 11:45
@miss-islington
Copy link
Contributor Author

@sobolevn and @gpshead: Status check is done, and it's a success ✅ .

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

Successfully merging this pull request may close these issues.

6 participants