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

JSON grammars #49 #96

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

JSON grammars #49 #96

wants to merge 2 commits into from

Conversation

panoply
Copy link

@panoply panoply commented Oct 27, 2021

As per #49 here is a converted list of TextMate Grammars from XML to JSON. In addition, --disable-extensions flag within extension launcher.

Reasoning for XML > JSON
JSON grammars are common for vscode. In addition, JSON grammars allow us to extended grammars in far less redundancy. We can also take advantage of additional vscode specific features like grammar injections with less verbosity.

Reasoning for --disable-extensions flag
Loading up an instance of vscode without this flag will result in installed extensions from running. It is allows for a quicker load time in debug mode.

Ensure extensions are disabled when running launch. It provides us with better oversight without conflicts.
XML grammars are great but they a verbose, use JSON format grammars.
@panoply
Copy link
Author

panoply commented Oct 27, 2021

Expect some additional amendments to the structure of the grammers, ie: improving token definitions and moving syntaxes into a pattern based architecture.

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.

1 participant