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

Build both PDF and web in texlive-action #8

Merged
merged 1 commit into from
Dec 2, 2023
Merged

Build both PDF and web in texlive-action #8

merged 1 commit into from
Dec 2, 2023

Conversation

utensil
Copy link
Contributor

@utensil utensil commented Dec 2, 2023

This PR fixes the inconsistency issue in #5 which was trying to accelerate LaTeX building but it only moved the PDF building into texlive-action (which is the key to the acceleration) but blueprint website is left out as I wasn't sure it will work well inside texlive-action .

Soon I realized that plastex (leanblueprint relies on it to generate the web version) will need a full texlive installation for more adavanced features, e.g. the support for adding commutative diagram to both the PDF and the web page requires rendering tikz-cd to PDF then SVG images for the web page , the inconsistency breaks functionalities like this.

After fixing an issue related to Python 3.11 compatibility, both PDF and web can be built in texlive-action, hence the PR. This will also be ~1 minute faster due to less duplicated installation.

@utensil utensil merged commit f890676 into main Dec 2, 2023
1 check passed
@utensil utensil deleted the unify-ci branch December 2, 2023 10:43
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.

1 participant