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 dockerfile highlighting #576

Merged
merged 3 commits into from
Sep 5, 2015
Merged

Conversation

JustinBeckwith
Copy link
Contributor

No description provided.

'keyword': {
pattern: /^\s*\b(ONBUILD|FROM|MAINTAINER|RUN|EXPOSE|ENV|ADD|COPY|VOLUME|USER|WORKDIR|CMD|ENTRYPOINT)\b\s/mi
},
'string': /("|')(\\\n|\\?.)*?\1/,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please be careful with the indentation. Prism uses tabs to indent, not spaces.

@Golmote
Copy link
Contributor

Golmote commented May 9, 2015

Thanks for contributing! Would you mind taking a look at my comments?
Also, you need to rename the JS file from prism-docker.js to prism-dockerfile.js, since the component is dockerfile.

},
'string': /("|')(\\\n|\\?.)*?\1/,
'comment': /#[^\n]+/,
'punctuation': /([:[\]{}\-,|>?]|---|\.\.\.)/
Copy link
Contributor

Choose a reason for hiding this comment

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

No need for the parentheses here.

@JustinBeckwith
Copy link
Contributor Author

Thanks for the awesome feedback. I updated the PR with all of the requested updates. Let me know if there's anything else!

@Golmote
Copy link
Contributor

Golmote commented May 20, 2015

@JustinBeckwith Sorry for the delay. Your updates look good to me. You did not add the LABEL keyword though, is it intended?
@LeaVerou, if it looks good to you too, feel free to merge this. I'll run the gulp task to generate the minified file if you don't have the time to do so.

@JustinBeckwith
Copy link
Contributor Author

That was an oversight :) Thanks again!

@egut
Copy link

egut commented Jun 8, 2015

Ohh exactly what I was looking for, any idea when it will be merged?

(As I'm currently using bower so its nifty if I don't have to modify files within bower components)

/BR

Golmote added a commit that referenced this pull request Sep 5, 2015
@Golmote Golmote merged commit e808352 into PrismJS:gh-pages Sep 5, 2015
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.

3 participants