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 XC syntax #2192

Merged
merged 2 commits into from
Mar 3, 2015
Merged

Added XC syntax #2192

merged 2 commits into from
Mar 3, 2015

Conversation

graymalkin
Copy link

XC has various keywords which aren't part of C++, so while the C++ syntax highlighting mostly works, there's a lot of cases where it doesn't.

I've forked the C/C++ textmate definition and have it working for XC's extra keywords.

See here for a preview.

@arfon
Copy link
Contributor

arfon commented Mar 3, 2015

👍 thanks @graymalkin

arfon added a commit that referenced this pull request Mar 3, 2015
@arfon arfon merged commit 600bca1 into github-linguist:master Mar 3, 2015
@graymalkin
Copy link
Author

Just curious, how long do changes to Linguist take to go live?

There's no rush -- just curious. 🍻

@arfon
Copy link
Contributor

arfon commented Mar 4, 2015

Just curious, how long do changes to Linguist take to go live?

Usually within a week or so. Hopefully we'll cut a new gem later this week.

@graymalkin
Copy link
Author

Ah, fantastic! Thanks arfon :)

@arfon arfon mentioned this pull request Mar 5, 2015
@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.

2 participants