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

warning as error default to OFF and enabled in CI #3814

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

JohanMabille
Copy link
Member

@JohanMabille JohanMabille commented Feb 11, 2025

Fixes #3791

The option MAMBA_WARNING_AS_ERROR has to be explictly set in the micromamba feedstock to catch it in this CI.
See conda-forge/micromamba-feedstock#241

@jjerphan jjerphan added the release::ci_docs For PRs related to CI or documentation label Feb 11, 2025
@JohanMabille JohanMabille merged commit 89abba3 into mamba-org:main Feb 11, 2025
34 checks passed
@JohanMabille JohanMabille deleted the warning_as_error branch February 11, 2025 14:05
Copy link
Member

@Hind-M Hind-M left a comment

Choose a reason for hiding this comment

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

Is it fixing #3791? IIUC, it's considering warnings as errors but it's not really fixing the errors reported?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release::ci_docs For PRs related to CI or documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

libmamba 2.0.5 fails to build on Fedora rawhide
4 participants