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

ci: require new fixtures be committed #28

Closed

buffalojoec wants to merge 2 commits into main from force-fixture-commits

Contributor

buffalojoec commented Nov 25, 2024 •

edited

Loading

Simply adds a CI job to ensure any time a test is changed, the new fixtures are generated and committed. Similar to CU benchmarking.

buffalojoec force-pushed the force-fixture-commits branch 3 times, most recently from 837f9f2 to 6d537ef Compare

November 25, 2024 08:38

buffalojoec added 2 commits

November 25, 2024 17:48


          fuzz: generate firedancer-compatible fixtures

707e33c


          ci: require new fixtures be committed

919834e

buffalojoec force-pushed the force-fixture-commits branch from 6d537ef to 919834e Compare

November 25, 2024 08:52

buffalojoec closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet