Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fmt/goto-defn: drop functionality in favor of LSP implementation #156

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

charlieegan3
Copy link
Collaborator

We are trying to standardize Rego developer env functionality on the Regal LS implementation to be able to offer a standardized experience to all Rego developers, regardless of their preferred editor.

Fmt on save and goto definition are now available as part of the language server and so should be accessed via that route instead.

We are trying to standardize Rego developer env functionality
on the Regal LS implementation to be able to offer a standardized
experience to all Rego developers, regardless of their preferred editor.

Fmt on save and goto definition are now available as part of the
language server and so should be accessed via that route instead.

Signed-off-by: Charlie Egan <charlie@styra.com>
@anderseknert anderseknert force-pushed the drop-lsp-functionality branch from 12c73ee to eff41e0 Compare April 22, 2024 17:14
Copy link
Member

@anderseknert anderseknert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would probably be good to add a note on how to disable / configure linting until we have a toggle for that, but LGTM!

@charlieegan3
Copy link
Collaborator Author

Yeah that makes sense, will do tomorrow.

@anderseknert anderseknert merged commit 635d8f0 into master Apr 23, 2024
4 checks passed
@anderseknert anderseknert deleted the drop-lsp-functionality branch April 23, 2024 09:04
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants