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

Replace languageId by languageSelector in all language feature registration functions #143610

Merged

Conversation

CGNonofr
Copy link
Contributor

It allows to register a language provider on more than one language at once.

This PR fixes microsoft/monaco-editor#2985

@ghost
Copy link

ghost commented Feb 22, 2022

CLA assistant check
All CLA requirements met.

@hediet
Copy link
Member

hediet commented Feb 22, 2022

This change has no runtime implications, it just affects typing and parameter names.

The suggested change looks reasonable.

@alexdima alexdima added this to the February 2022 milestone Feb 22, 2022
@alexdima alexdima merged commit 071578a into microsoft:main Feb 22, 2022
@alexdima
Copy link
Member

Thank you!

@CGNonofr
Copy link
Contributor Author

btw, shouldn't it appear in the monaco-editor 0.33 changelog?

@hediet
Copy link
Member

hediet commented Mar 14, 2022

Sorry, we missed that!
It would be awesome if you could file a PR to monaco-editor to update the changelog.

Unfortunately we cannot update 0.33.0 on NPM anymore.

@CGNonofr
Copy link
Contributor Author

👍 => microsoft/monaco-editor#3022

@github-actions github-actions bot locked and limited conversation to collaborators Apr 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request] Allow to register a language provider using a language selector instead of a single language
4 participants