-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Check if a fatal warning issued in typer is silenced, before converting it into an error #18089
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a number of issues with this.
First the premise: I don't see how this is doing anything before converting it to an error, if anything it looks like it's converting warnings to errors before running through Wconf.
Your test doesn't seem to capture the intent of the original 17741 and the issue 17735 doesn't have a test case either. Your 17741 seems to be triggering a auto-apply-insertion warning, which is correctly suppressed by @nowarn
. So it's nothing to do with -Wnonunit-statement
, which is also missing from the test case anyways.
…ng it into an error closes lampepfl#17741 closes lampepfl#17735
bb1fd4a
to
ad45c3e
Compare
Converting warnings to errors before checking This change is supposed to address the fact that when using The bug had nothing to do with |
@dwijnand Have you had some time to read my last comment? (It might have gotten buried in your notifications) |
closes lampepfl#17741
closes lampepfl#17735