Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Remove toolkit functions deprecated in 3.1#15995
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| @@ -863,7 +863,7 @@ Use the standard library's docstring manipulation tools instead, such as | |||
| These are considered internal and will be removed from the public API in a | |||
| future version. | |||
| - `.projections.process_projection_requirements` | |||
| - ``projections.process_projection_requirements`` | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
these should technically be in a separate PR but I'm not going to be nitpicky with that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
If one wants to be very strict yes, but it's just changing link to literal, which can be done anytime is not strictly related to the deprecation of that function. I saved the hassle of cherry-pickling parts of the file. 😄
PR Summary
This pulls out parts of#15890 to make that PR more managable and easier to debug the test failures.