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

Separate cases in stringify and restify with greater precision #12284

Merged
merged 14 commits into from
Apr 23, 2024

Conversation

picnixz
Copy link
Member

@picnixz picnixz commented Apr 15, 2024

This is the last part of the refactorization of those functions where I moved some code around and refactor some inner cases (with details).

@picnixz picnixz changed the title [refactor] separate stringify and restify cases more precisely [refactor] separate stringify and restify cases more precisely [part 2e] Apr 15, 2024
@AA-Turner AA-Turner force-pushed the cleanup/util-2e-typing-refactor branch from 2e5ea02 to 866a2e1 Compare April 23, 2024 05:12
@picnixz
Copy link
Member Author

picnixz commented Apr 23, 2024

@AA-Turner Since you are working on my PR, I'll let you handle whatever changes you want to do (so that we don't have conflicts if I push something else). Feel free to ping me if you have questions!

EDIT: It also appears that CPython 3.13 changed the representation somewhere causing an autodoc test to fail (it happened before you merged my PRs).

@AA-Turner AA-Turner changed the title [refactor] separate stringify and restify cases more precisely [part 2e] Separate cases in stringify and restify with greater precision Apr 23, 2024
@chrisjsewell
Copy link
Member

Heya, my one comment here is; restify is only ever used within sphinx/ext/autodoc. I get that this would be a breaking change, but it really should be moved there, since it has nothing to do with core Sphinx

@AA-Turner AA-Turner merged commit 1ff9adf into sphinx-doc:master Apr 23, 2024
21 of 23 checks passed
@picnixz picnixz deleted the cleanup/util-2e-typing-refactor branch April 23, 2024 10:24
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 24, 2024
@AA-Turner AA-Turner added this to the 7.4.0 milestone Jul 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants