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

tests: Use typst-test #470

Merged
merged 1 commit into from
Feb 8, 2024
Merged

tests: Use typst-test #470

merged 1 commit into from
Feb 8, 2024

Conversation

johannes-wolf
Copy link
Member

@johannes-wolf johannes-wolf commented Jan 17, 2024

Waiting for typst-test to support non-zero return codes on failure.
Ticket: tingerrr/typst-test#6

Copy link

@tingerrr tingerrr left a comment

Choose a reason for hiding this comment

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

I've tested the reworked implementation on my own repo with a similar (blatantly stolen) CI workflow and locally.

I think it's ready to use now.

.github/workflows/ci.yml Show resolved Hide resolved
@johannes-wolf
Copy link
Member Author

johannes-wolf commented Feb 7, 2024

@fenjalien Can we update & merge this?

Copy link
Member

@fenjalien fenjalien left a comment

Choose a reason for hiding this comment

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

Absolutely :)

@johannes-wolf johannes-wolf force-pushed the use-typst-test branch 4 times, most recently from c9ddef4 to cce4277 Compare February 8, 2024 14:17
@johannes-wolf johannes-wolf merged commit e76a44d into master Feb 8, 2024
1 check passed
@johannes-wolf johannes-wolf deleted the use-typst-test branch February 8, 2024 14:23
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.

3 participants