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

Remove spurious whitespace in workspace/executeCommand handler #28

Merged
merged 1 commit into from Jun 8, 2022

Conversation

keynmol
Copy link
Contributor

@keynmol keynmol commented Jun 8, 2022

I was getting No method by the name 'workspace/executeCommand' is found. even though I registered all the providers correctly.

This seems like a good candidate for a reason why :)

@baronfel
Copy link
Contributor

baronfel commented Jun 8, 2022

OMFG that explains so much about my own experiments with workspace commands.

@baronfel baronfel merged commit 85d9eb7 into ionide:main Jun 8, 2022
@keynmol keynmol deleted the patch-1 branch June 8, 2022 15:18
@keynmol
Copy link
Contributor Author

keynmol commented Jun 8, 2022

Thanks for the Fastest Merge In The West!

@baronfel
Copy link
Contributor

baronfel commented Jun 8, 2022

I'll try to get a release out later today US Eastern Time with this fix.

@keynmol
Copy link
Contributor Author

keynmol commented Jun 8, 2022

👍 even though I'm using it as a vendored dependency, it's a good idea to not have a stable version anywhere with this bug :D

@baronfel
Copy link
Contributor

baronfel commented Jun 8, 2022

Selfishly I want it available for FSAC. We don't use a vendored dependency in part to encourage me to quickly make releases here :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants