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

LaTeX: add \sphinxnotetitle et al. to sphinxlatexadmonitions.sty #11266

Merged
merged 3 commits into from Mar 26, 2023

Conversation

jfbu
Copy link
Contributor

@jfbu jfbu commented Mar 26, 2023

This refactoring in sphinxlatexadmonitions.sty also fixes #11264.

@jfbu jfbu added this to the 6.2.0 milestone Mar 26, 2023
@jfbu
Copy link
Contributor Author

jfbu commented Mar 26, 2023

Seems pip itself causes the failed testing with Python 3.12 here and here

In passing I reverted some deprecation I had added at 6bf4059 to some
internals of the sphinxadmonition environment, as it does not look that
pressing anyhow and the redundant namings of two colors in particular
may nevertheless prove useful to user \renewcommand's, as they avoid to
have to refer to \spx@noticetype.
@jfbu
Copy link
Contributor Author

jfbu commented Mar 26, 2023

I have pushed to master 321f886 which avoids failed test due to pip currently still using "onerror" which has been deprecated by Python 3.12-dev in favor of "onexc" in shutil.rmtree(). I rebased this on top of that and will merge once testing completes.

@jfbu jfbu merged commit 7144f49 into sphinx-doc:master Mar 26, 2023
22 checks passed
@jfbu jfbu deleted the latex_admonitions branch March 26, 2023 22:14
jfbu added a commit to jfbu/sphinx that referenced this pull request Mar 27, 2023
Follow-up to sphinx-doc#11266.  Using \sphinxstyle prefix looked a posteriori
better, for coherency, although it is a bit long.
@jfbu
Copy link
Contributor Author

jfbu commented Mar 27, 2023

The added LaTeX commands were renamed at follow-up #11267 for coherence with previous existing ones.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 27, 2023
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.

LaTeX: missing space before colon after "Voir aussi" for seealso directive in French
1 participant