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.

Sign up for GitHub

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

Jump to bottom

LSP: Adding CodaActionProvider API. #6770

Merged
merged 1 commit into from Dec 1, 2023
Merged

LSP: Adding CodaActionProvider API. #6770

merged 1 commit into from Dec 1, 2023

Conversation

dbalek
Copy link
Contributor

@dbalek dbalek commented Nov 30, 2023

Adding CodaActionProvider API, to allow for creation of code actions outside of the java.lsp.server module.

@dbalek dbalek added LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests labels Nov 30, 2023
@dbalek dbalek added this to the NB21 milestone Nov 30, 2023
@dbalek dbalek self-assigned this Nov 30, 2023
@dbalek dbalek requested a review from sdedic November 30, 2023 12:13
lahodaj
lahodaj approved these changes Nov 30, 2023
View reviewed changes
Copy link
Contributor

@lahodaj lahodaj left a comment

Choose a reason for hiding this comment

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

Looks sensible to me.

@dbalek dbalek merged commit 8260302 into apache:master Dec 1, 2023
35 checks passed
@dbalek dbalek deleted the dbalek/lsp-code-action-provider branch December 1, 2023 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MartinBalin MartinBalin MartinBalin approved these changes

@lahodaj lahodaj lahodaj approved these changes

@jhorvath jhorvath Awaiting requested review from jhorvath

@sdedic sdedic Awaiting requested review from sdedic

Assignees

@dbalek dbalek

Labels
LSP [ci] enable Language Server Protocol tests VSCode Extension [ci] enable VSCode Extension tests
Projects
None yet
Milestone
NB21
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dbalek @MartinBalin @lahodaj