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

Add Racket syntax #1885

Merged
merged 1 commit into from
Oct 3, 2021
Merged

Add Racket syntax #1885

merged 1 commit into from
Oct 3, 2021

Conversation

jubnzv
Copy link
Contributor

@jubnzv jubnzv commented Oct 3, 2021

Add syntax files for Racket language.

It is based on Racket syntax highlighting plugin, which have more than 10K downloads at packagecontrol.

For the test, I used some old MIT-licensed code written by me.

Closes #1884

@sharkdp
Copy link
Owner

sharkdp commented Oct 3, 2021

Thank you very much for your contribution!

I think the output file for the syntax highlighting tests is missing?

@jubnzv
Copy link
Contributor Author

jubnzv commented Oct 3, 2021

@sharkdp Oops, I added the missing file.

Copy link
Owner

@sharkdp sharkdp left a comment

Choose a reason for hiding this comment

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

@Enselic should we wait with adding new syntaxes while your syntax loading improvements are still in progress?

@Enselic
Copy link
Collaborator

Enselic commented Oct 3, 2021

@sharkdp Nope, no need to wait, so please go ahead.

(I never (deliberately) merge anything that blocks others or for technical reasons blocks us from making a release)

@sharkdp sharkdp merged commit d04a83d into sharkdp:master Oct 3, 2021
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.

Syntax highlighting for Racket
3 participants