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.

Sign up for GitHub

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

Jump to bottom

Update CI checks to include running tests #38

Merged
merged 6 commits into from
Jan 29, 2025
Merged

Update CI checks to include running tests #38

merged 6 commits into from
Jan 29, 2025

Conversation

mikewesthad
Copy link
Member

@mikewesthad mikewesthad commented Jan 29, 2025 •
edited
Loading

Update the newly added CI check to include the snapshot test setup

Test plan:

  • ab9a5f3 features an intentional error in the extrusion logic and fails CI
  • 8b3ca27 reverts the intentional error and passes CI

@mikewesthad mikewesthad merged commit 95503ec into main Jan 29, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mikewesthad