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

Check if tokens exists and return #1195

Merged

Conversation

mAAdhaTTah
Copy link
Member

If the language was autoloaded, the grammar may not exist yet.
We need to check if it exists before we attempt to modify it.

See here for details: 70cb472

If the language was autoloaded, the grammar may not exist yet.
We need to check if it exists before we attempt to modify it.
@mAAdhaTTah
Copy link
Member Author

mAAdhaTTah commented Sep 30, 2017

@Golmote All of the plugins except this one check the existence of grammar first. This PR is good to go. Can you merge this tomorrow? #hacktoberfest

@Golmote
Copy link
Contributor

Golmote commented Sep 30, 2017

I don't think I'll have time for Prism until next weekend. This PR is on my to-check list though

mAAdhaTTah added a commit to intraxia/wp-gistpen that referenced this pull request Sep 30, 2017
mAAdhaTTah added a commit to intraxia/wp-gistpen that referenced this pull request Sep 30, 2017
mAAdhaTTah added a commit to intraxia/wp-gistpen that referenced this pull request Oct 1, 2017
mAAdhaTTah added a commit to intraxia/wp-gistpen that referenced this pull request Oct 1, 2017
mAAdhaTTah added a commit to intraxia/wp-gistpen that referenced this pull request Oct 1, 2017
@Golmote Golmote merged commit ed8bdb5 into PrismJS:gh-pages Oct 10, 2017
@Golmote
Copy link
Contributor

Golmote commented Oct 10, 2017

Ok, merged!

@mAAdhaTTah mAAdhaTTah deleted the bugfix/show-invisibles-missing-grammar branch October 10, 2017 12:02
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.

None yet

2 participants