-
Notifications
You must be signed in to change notification settings - Fork 338
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
Fix #2248 #2249
Fix #2248 #2249
Conversation
Hm this is bad the test is supposed to fail |
I think the conda version wasn't released on conda-forge yet, because of a conda-build bug. |
Ah, I looked at the wrong place! https://github.com/jonashaag/mamba/actions/runs/4010134326/jobs/6886285496 |
Please squash merge |
Hmm not sure what's going on |
Yeah, this is ongoing |
That's good because we haven't fixed compatibility yet! :-D |
Isn’t the last commit just another way to write exactly the same thing? |
OK, I think I finally understand what's going on ... running |
Another approach would’ve been to set $HOME to a temporary directory for the test. |
yes, that's true and is also something we do in the micromamba tests. I'd be fine with merging and releasing this though :) |
Closes #2248