Add automatic detection update of new patterns in tinytex #13619
+387
−6
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cderv
changed the title
Collaborator
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
Gracefully exit with warning if TinyTeX daily release download fails. This prevents workflow failure when daily release hasn't been published yet.
- Add first_run detection in compare step - Create GitHub Actions summary on first run with: - TinyTeX commit SHA and link to R/latex.R - Pattern and category counts - Baseline establishment message - Update Exit if unchanged to not exit on first run - Add cache update step for pattern changes - Update Save cache condition to include first_run - Update Summary step to handle all three cases
Collaborator
Author
|
Test on my fork and leads to create issue like this On first run, there will be no diff and an initialization leading to some summary step |
cderv
force-pushed
the
tinytex/check-pattern-daily
branch
from
d1a349c to
d6a0895
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Quarto LaTeX engine can automatically install missing packages and fonts because it parsed log file.
The parsing happens through known error using regex patterns.
Quarto leverages TinyTeX mainly, and experience in tinytex R package.
On tinytex side, we did export the patterns through
regex.jsonfile so we can sync between project. We haven't done so.It is like the
tinitexCLI tool that is supposed to have all the clever latexmk logic in one tool instead of duplicating in here and in tinytex.First step is using the
regex.json. This PR adds a workflow file to check daily for new entries, and open an issue for us to look at so that we don't miss any update in the future