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

Lthooks new showhook 2 #1249

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

Conversation

jlaurens
Copy link
Contributor

  • tests for arguments
  • show test file frozen for the 2023-06-01 release
  • show test file for the latest release, will be used to change \ShowHook in a forthcoming PR

No date change, no code change

2 commits, the first one is the same as in PR #1248

Pull requests in this repository are intended for LaTeX Team members only.

Internal housekeeping

Status of pull request

  • Feedback wanted
  • Under development
  • Ready to merge

Checklist of required changes before merge will be approved

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

@jlaurens jlaurens force-pushed the lthooks-new-showhook-2 branch 2 times, most recently from 4267744 to e366eb5 Compare January 23, 2024 08:21
@jlaurens
Copy link
Contributor Author

Test names conform to other frozen tests names.

@jlaurens jlaurens mentioned this pull request Jan 23, 2024
6 tasks
@jlaurens jlaurens force-pushed the lthooks-new-showhook-2 branch 3 times, most recently from cbd4ce2 to e71fe05 Compare January 23, 2024 11:32
@jlaurens jlaurens marked this pull request as draft February 2, 2024 07:26
@josephwright
Copy link
Member

I'm not sure I see the difference between this PR and #1248 in a functional sense: could you explain a bit more?

@jlaurens
Copy link
Contributor Author

jlaurens commented Jun 4, 2024

PR 1248 has some tests frozen to the 2020 release whereas this one has some more tests frozen to the 2023 release.

@josephwright josephwright self-requested a review June 4, 2024 14:11
@jlaurens
Copy link
Contributor Author

jlaurens commented Jun 4, 2024

Maybe one dedicated branch for this PR and the forthcoming ones before everything is organized into bigger commits and merged to develop

@josephwright josephwright marked this pull request as ready for review June 4, 2024 16:35
@josephwright
Copy link
Member

@jlaurens From what I can see, this PR should be merged as it includes everything from #1248. Assuming that's correct, I can look to rebase tomorrow and get merged.

- `lthooks-show-2020-10-01.lvt` is frozen at the 2020-10-01 release,
  no arguments in hooks
- `update-lthooks-tests.sh` includes the above test
- `lthooks-show.sty` is a library of test helpers. They will be used by
  forthcoming tests, as well as completed.
- tests for arguments
- show test file frozen for the 2023-06-01 release
- show test file for the latest release, will be used to change `\ShowHook` in a forthcoming PR

No date change, no code change
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