Set options in IDE to more closely match CLI #4374
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This makes IDE and CLI resource counts more likely to match. In particular, it makes it so that the Boogie file generated when running the language server with the `--bprint` flag matches the Boogie file generated when running the CLI with the `/print` Boogie flag, except for some comments that don't seem to affect RU counts. This does not, unfortunately, always make the RU counts presented in the IDE match those given by the CLI. I'm not yet sure where the additional discrepancy is coming from.
keyboardDrummer
approved these changes
Aug 3, 2023
keyboardDrummer
pushed a commit
to keyboardDrummer/dafny
that referenced
this pull request
Sep 15, 2023
This makes IDE and CLI resource counts more likely to match. In particular, it makes it so that the Boogie file generated when running the language server with the `--bprint` flag matches the Boogie file generated when running the CLI with the `/print` Boogie flag, except for some comments that don't seem to affect RU counts. This does not, unfortunately, always make the RU counts presented in the IDE match those given by the CLI. I'm not yet sure where the additional discrepancy is coming from. By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.
keyboardDrummer
pushed a commit
that referenced
this pull request
Sep 19, 2023
This makes IDE and CLI resource counts more likely to match. In particular, it makes it so that the Boogie file generated when running the language server with the `--bprint` flag matches the Boogie file generated when running the CLI with the `/print` Boogie flag, except for some comments that don't seem to affect RU counts. This does not, unfortunately, always make the RU counts presented in the IDE match those given by the CLI. I'm not yet sure where the additional discrepancy is coming from. By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This makes IDE and CLI resource counts more likely to match. In particular, it makes it so that the Boogie file generated when running the language server with the
--bprintflag matches the Boogie file generated when running the CLI with the/printBoogie flag, except for some comments that don't seem to affect RU counts.This does not, unfortunately, always make the RU counts presented in the IDE match those given by the CLI. I'm not yet sure where the additional discrepancy is coming from.
By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.