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

Shall we introduce -Werror flag? #11844

Open
radeusgd opened this issue Dec 11, 2024 · 0 comments
Open

Shall we introduce -Werror flag? #11844

radeusgd opened this issue Dec 11, 2024 · 0 comments
Assignees
Labels
-compiler p-low Low priority x-new-feature Type: new feature request

Comments

@radeusgd
Copy link
Member

radeusgd commented Dec 11, 2024

I've just found out (I think not for the first time) that we have code in Enso Standard Library that yields compiler warnings.

I think it would be useful if we could make our CI fail if there are any warnings in one of our Standard Libraries (or tests as well). Of course by default warnings should not be fatal, but if we can make them fatal upon request, we could ensure that we will not accidentally merge PRs with warnings in the code.

Of course the name of the CLI argument may be something else, I just used -Werror as a well-known example.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-compiler p-low Low priority x-new-feature Type: new feature request
Projects
Status: New
Development

No branches or pull requests

3 participants