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

Am/chore/fix deps pinning #932

Merged
merged 2 commits into from
Feb 29, 2024
Merged

Conversation

IceTDrinker
Copy link
Member

Fixes #931

Used allow everywhere as some changes would likely be semver breaking even if some stuff is unused

@cla-bot cla-bot bot added the cla-signed label Feb 28, 2024
@IceTDrinker
Copy link
Member Author

@slab-ci cpu_test
@slab-ci cpu_integer_test
@slab-ci cpu_multi_bit_test
@slab-ci cpu_wasm_test
@slab-ci csprng_randomness_testing

Copy link

@slab-ci cpu_fast_test

@IceTDrinker
Copy link
Member Author

Already launched the tests as they are super long and the breakage is very bad

@IceTDrinker
Copy link
Member Author

I left my assert of the forward compat to test 🫠

@IceTDrinker IceTDrinker force-pushed the am/chore/fix-deps-pinning branch from 09beb57 to 84e61c2 Compare February 28, 2024 19:02
Copy link

@slab-ci cpu_fast_test

@IceTDrinker
Copy link
Member Author

@slab-ci cpu_test
@slab-ci cpu_integer_test
@slab-ci cpu_multi_bit_test
@slab-ci cpu_wasm_test
@slab-ci csprng_randomness_testing

@IceTDrinker
Copy link
Member Author

@mayeul-zama the review is to spot stuff that may look odd, CI is already in progress so no need to approve, it would just re run it

Copy link
Contributor

@soonum soonum left a comment

Choose a reason for hiding this comment

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

Everything looks fine to me. Let's merge.

@IceTDrinker IceTDrinker merged commit b80e09f into release/0.4.x Feb 29, 2024
20 checks passed
@IceTDrinker IceTDrinker deleted the am/chore/fix-deps-pinning branch February 29, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants