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

Move localeLoader to monaco-languageclient #672

Closed
wants to merge 1 commit into from
Closed

Conversation

kaisalmen
Copy link
Collaborator

It makes more sense there, because this is not a specific functionality of the wrapper and could then be used without it. WDYT?

@kaisalmen kaisalmen requested a review from CGNonofr as a code owner June 4, 2024 12:26
@CGNonofr
Copy link
Collaborator

CGNonofr commented Jun 4, 2024

I'm not sure about the exact architectures of the projects, it doesn't seem really related to the languageclient package to me

@kaisalmen
Copy link
Collaborator Author

I'm not sure about the exact architectures of the projects, it doesn't seem really related to the languageclient package to me

It is more a utility function for monaco-vscode-api, but you don't need the wrapper for it. But, if you think it should not go to the client, I will leave it where it currently is and close this PR without merging.

Copy link
Collaborator

@CGNonofr CGNonofr left a comment

Choose a reason for hiding this comment

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

Change is simple, so I'll let you decide what you think is the best

@kaisalmen
Copy link
Collaborator Author

@CGNonofr I changed my mind and let it stay where it is.

@kaisalmen kaisalmen closed this Jun 4, 2024
@kaisalmen kaisalmen deleted the mv-localeLoader branch June 4, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants