-
Notifications
You must be signed in to change notification settings - Fork 0
Quarto checks #99
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
Merged
Merged
Quarto checks #99
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
28202ca
🦺 Refactor: add quarto_checks argument
sayalaruano 5b4f1c7
🐛 Fix: Correct validation for revelajs and jupyter notebooks
sayalaruano a53a6f8
🎨 Format: add config_path parameter and change the strcuture of the f…
sayalaruano d271f52
🎨 Format: add black format
sayalaruano 49027e1
🎨 Format: order main imports
sayalaruano 82fbc1e
🐛 Fix: fix typo for quarto_checks
sayalaruano 4060fa4
🐛 Fix: add quarto_checks into the report_generator funcrtion
sayalaruano 2caacce
🎨 Format: apply black format and fix quarto_checks typo
sayalaruano b30128b
🐛 Fix: fix quarto_checks typo in quarto_review.py
sayalaruano 97a70bd
🐛 Fix quarto_checks typo
sayalaruano b82fae1
🐛 Fix: add -qt_checks parameter into the CICD file
sayalaruano d2d2b40
🐛 Fix: add html suffix for jupyter notebook for initial validation
sayalaruano eb4e626
:bug: fix import and specify that GUI execution checks tinytex and ch…
enryH a00746c
✨ Feat: add output_dir as an argument
sayalaruano 3f23318
🐛 Fix: correct output_dir typo
sayalaruano 4264171
🐛 Fix: add -qt_checks parameter in the CICD processs for the pdf repo…
sayalaruano File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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.
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.
so here it is needed to install tinytex on the fly, right? but just not in the docker image?
It's good that this has to be activated, so that updates do not just happen. but we should document this as well.