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

Added function to register and clear external schema in the language service Interface #187

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rahulbanerjee26
Copy link
Contributor

This pull request adds a couple of functions to the Json Language Service Interface

  • Function to register a schema
  • Function to clear a previously registered schema.

Right now, if a user wants to register or clear schemas, the only way to do it is by using configure. Since configure overwrites previously registered schemas, this would require the user to maintain a list of schemas and update it accordingly and use it along with configure.

I noticed JSONSchema service has some utility functions that could make the above process easier. Please let me know your thoughts or if I misunderstood something and if there is currently a way to register/clean schemas without having to maintain a list of schemas.

@rahulbanerjee26
Copy link
Contributor Author

@microsoft-github-policy-service agree

@rahulbanerjee26
Copy link
Contributor Author

@aeschli Could you please take a look when convenient?

@aeschli
Copy link
Contributor

aeschli commented Apr 25, 2023

@rahulbanerjee26 I'd rather avoid extra API just for convenience. In the end it's API to maintain.

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.

2 participants