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 G-Code grammar #4778

Merged
merged 1 commit into from
Jan 23, 2020
Merged

Replace G-Code grammar #4778

merged 1 commit into from
Jan 23, 2020

Conversation

lildude
Copy link
Member

@lildude lildude commented Jan 23, 2020

Description

CI is currently failing because the original G-Code grammar has gone MIA as the user appears to have deleted their account and all code from GitHub.

This PR replaces the grammar with what appears to be a newer and much more actively maintained grammar from https://github.com/appliedengdesign/vscode-gcode-syntax.

The highlighting isn't identical, but given the replacement is newer and more actively maintained, I think it's probably more accurate.

Checklist:

  • I am changing the source of a syntax highlighting grammar

/cc FYI @mikecentola as you are the maintainer of the grammar we're now using. This will become the grammar used to highlight G-Code files on GitHub.com when I make the next release after this PR has been merged.

The original went MIA without warning.
@Alhadis
Copy link
Collaborator

Alhadis commented Jan 23, 2020

as the user appears to have deleted their account and all code from GitHub.

You damn well better be glad I'm not going anywhere then. 😉

@lildude lildude merged commit d139b53 into master Jan 23, 2020
@lildude lildude deleted the lildude/replace-gcode-grammar branch January 23, 2020 07:41
@mikecentola
Copy link

@lildude Glad to help! I will continue to maintain the extension and let me know if there's any suggestions!

@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants