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

Gh1585 allowing \fpeval in array preamble #1586

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

FrankMittelbach
Copy link
Member

Internal housekeeping

Status of pull request

Ready to merge

Checklist of required changes before merge will be approved

  • Test file(s) added
  • Version and date string updated in changed source files
  • Relevant \changes entries in source included
  • Relevant changes.txt updated
  • [n/a] Rollback provided (if necessary)?
  • ltnewsX.tex (and/or latexchanges.tex) updated

There is one surprising test suite diff for tagging-002-longtable. That needs checking and verification.

@FrankMittelbach
Copy link
Member Author

I've done some minimal adjustment to longtable, to make it work with the change in array. It could probably be simplified by using @startpbox@preamble and then keeping @startpbox for the action rather than having @startpbox and @startpboxaction with the same top-level definitions. But that might mean more checking on other packages how they use @startpbox and @mkpream. So I'm not sure it is worth the effort. @davidcarlisle does it looks okay to you?

required/tools/array.dtx Outdated Show resolved Hide resolved
required/tools/array.dtx Outdated Show resolved Hide resolved
@FrankMittelbach FrankMittelbach requested review from davidcarlisle and removed request for davidcarlisle December 21, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants