Skip to content

chore: don't run tests with different typst versions#173

Merged
ptsouchlos merged 2 commits intoptsouchlos:mainfrom
jkhsjdhjs:ci/remove-typst
Mar 18, 2026
Merged

chore: don't run tests with different typst versions#173
ptsouchlos merged 2 commits intoptsouchlos:mainfrom
jkhsjdhjs:ci/remove-typst

Conversation

@jkhsjdhjs
Copy link
Contributor

Tytanic uses its own baked-in typst version anyway, so testing with multiple typst versions doesn't make any difference, as typst is not used at all by the testrunner. The previous behavior of installing different typst versions was misleading, since they were not used at all. Furthermore, the docs were only built on the latest typst version anyway. Thus, typst only needs to be installed to build the docs.

Finally, the Justfile is adjusted to run the tests treating warnings as errors, so issues like fonts not being found are detected properly.

Tytanic uses its own baked-in typst version anyway, so testing with
multiple typst versions doesn't make any difference. Furthermore, the
docs were only built on the latest typst version anyway. Thus, typst
only needs to be installed to build the docs.
Copy link
Owner

@ptsouchlos ptsouchlos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me, thanks!

@ptsouchlos ptsouchlos merged commit 230ae11 into ptsouchlos:main Mar 18, 2026
2 checks passed
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