Skip to content

Conversation

meithecatte
Copy link
Contributor

This is necessary to use some TeX features, such as TikZ's graph drawing library.

As there isn't a reasonable way of detecting the necessity of using lualatex, the --latex-compiler option is now also exposed through config.yml, to make it possible to export the package if the statement only compiles with the right latex compiler.

Copy link
Member

@MasloMaslane MasloMaslane left a comment

Choose a reason for hiding this comment

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

Could you add some tests? For example to verify if --latex-compiler overrides config

This is necessary to use some TeX features, such as TikZ's graph drawing
library.

As there isn't a reasonable way of detecting the necessity of using
lualatex, the --latex-compiler option is now also exposed through
config.yml, to make it possible to export the package if the statement
only compiles with the right latex compiler.
@meithecatte
Copy link
Contributor Author

Thanks for pushing me to add tests, turns out I slightly broke the thing in a small last-minute refactor. I added what I'd consider to be a reasonable set of tests – let me know if you see any other cases we should be testing.

When I'm running the tests locally, I'm getting some failures around the run command that seem to be spurious. I hope everything will pass on CI...

Copy link
Member

@MasloMaslane MasloMaslane left a comment

Choose a reason for hiding this comment

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

I’ll merge after the tests pass. Don’t worry about failing tests in run command, they sometimes fail and i didn’t have time to fix them

@MasloMaslane MasloMaslane merged commit 152dec5 into sio2project:main Jan 30, 2025
8 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