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

no-verify should imply --allow-warnings #489

Open
kjx opened this issue Aug 24, 2024 · 4 comments
Open

no-verify should imply --allow-warnings #489

kjx opened this issue Aug 24, 2024 · 4 comments

Comments

@kjx
Copy link

kjx commented Aug 24, 2024

no-verify should imply--allow-warnings

cos if I say dafny run --no-verify I want to run it WITHOUT verification.
If I'm about do to that, I likely don't care about warnings, do i?

@keyboardDrummer
Copy link
Member

If I'm about do to that, I likely don't care about warnings, do i?

I don't know. Don't you? Maybe you just want to skip the thing that is time consuming.

@kjx
Copy link
Author

kjx commented Aug 26, 2024

It's not that I mind seeing the warnings:
but without --allow-warnings then the thing won't run if there's even one warning...

@keyboardDrummer
Copy link
Member

keyboardDrummer commented Aug 26, 2024

I think you could argue that the default for dafny run should be --allow-warnings=true, but I'm careful with changing options that might lead to less safe behavior, and having different defaults across different commands is tricky as well.

@kjx
Copy link
Author

kjx commented Aug 26, 2024

I could, but I wouldn't.

although we don't seem to have a native interpreter, dafny run means... to run the program. Usually that means: run only if verified. If it's doing that, it should certainly report the warnings.

but we've said "--no-verify" so we don't want it to be verified.
so we'd better be doing development, aka testing.
at which point, I don't see why I wouldn't want to run the program, whatever warnings I've got.

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

No branches or pull requests

2 participants