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

Allow compilation errors to surface as diagnostics #745

Open
anderseknert opened this issue May 26, 2024 · 0 comments
Open

Allow compilation errors to surface as diagnostics #745

anderseknert opened this issue May 26, 2024 · 0 comments

Comments

@anderseknert
Copy link
Member

While the jury is still out on whether things like strict mode checks should be adopted as linter rules, I think the editing experience would benefit from having compiler errors displayed inline rather than having most issued displayed in the editor, while others only are surfaced when "check on save" hits, or however the user chooses to compile the policy outside of the editor. It would be nice if compilation errors, including strict mode violations, got reported inline where they happened, which would more or less eliminate the need for running the check command out of band.

While compiler errors should be considered diagnostics, we should still maintain a separation of these errors from parser errors and linting violations, similar to how we already do for parser errors and linting violations.

@anderseknert anderseknert changed the title Allow optional compilation to diagnostics Allow compilation errors to surface as diagnostics May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: TODO
Development

No branches or pull requests

1 participant