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

Ci mamba fix #2762

Closed
wants to merge 27 commits into from
Closed

Conversation

Vipul-Cariappa
Copy link
Contributor

Builds on top of #2758

@Vipul-Cariappa
Copy link
Contributor Author

@certik, as I can't write to that branch, I will be using this branch.

@Vipul-Cariappa
Copy link
Contributor Author

I had to skip the test_bit_length integration test from llvm and llvm_jit. I have commented about it on an issue related to bit_length. see #1836 (comment).

@Vipul-Cariappa
Copy link
Contributor Author

@certik, this might be a simpler fix than #2758.
Only had to skip test_bit_length integrations test, compared to skipping both test_llvm and the integration test.
And we need not pin every single package. I have only pined the necessary packages, and most probably conda will install the newest available versions of the other packages.

@Vipul-Cariappa Vipul-Cariappa mentioned this pull request Jul 8, 2024
@certik
Copy link
Contributor

certik commented Jul 8, 2024

Fixed in #2764.

@certik certik closed this Jul 8, 2024
@Vipul-Cariappa Vipul-Cariappa deleted the ci_mamba_fix branch July 9, 2024 00:39
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