-
-
Notifications
You must be signed in to change notification settings - Fork 267
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
Extend firstaid for cleveref for gh1560 #1562
Conversation
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.
modulo the fact that I think this needs to be based on main for the hotfix, I think this is fine.
% \begin{macrocode} | ||
\newif\iftag@ |
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.
I'm surprised that \newif doesn't bark, when the conditional aready exists, but it looks like that in fact the case. Not sure how god that is, but then it must have been that way forever.
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.
Yes, it comes from plain and plain doesn't check anything :)
@@ -1,3 +1,7 @@ | |||
2024-11-24 Ulrike Fischer <[email protected]> | |||
|
|||
* latex2e-first-aid-for-external-files.dtx: add iftag@-test to cleveref firstaid for issue #1560 |
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.
add \iftag@ test ...
(but not important)
* add cleveref firstaid for issue #1560 * update hook tests
Internal housekeeping
Status of pull request
Checklist of required changes before merge will be approved
\changes
entries in source includedchanges.txt
updatedltnewsX.tex
(and/orlatexchanges.tex
) updated