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

adding system verilog language #640

Merged
merged 2 commits into from Sep 16, 2015
Merged

adding system verilog language #640

merged 2 commits into from Sep 16, 2015

Conversation

a-rey
Copy link
Contributor

@a-rey a-rey commented Jul 11, 2015

Hello!

I use some hardware description languages in a website of mine and was sad to see they are not all supported by Prism. Here is my proposed Prism support for the System Verilog and Verilog languages. Since System Verilog is just an extension of Verilog, I combined both into one. The file can be used to highlight either language. I also added an example for you guys to take a look at if you are not familiar with the syntax. This handout is a great reference for syntax rules. Let me know if there are any issues!

Here is some sample code:
screen shot 2015-07-11 at 5 05 09 pm

@@ -328,6 +328,10 @@ var components = {
"require": "clike",
"owner": "chrischares"
},
"system_verilog": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Would you mind naming the language with a dash instead of an underscore?
Or maybe just name it verilog, since this is the base language?

@Golmote
Copy link
Contributor

Golmote commented Jul 11, 2015

Thanks for contributing! Please take a look at my comments.

@a-rey
Copy link
Contributor Author

a-rey commented Jul 12, 2015

Renamed the component to verilog. Added more info to components.js and prism-verilog.html to note the crossover between the 2 languages. Made the requested changes and updated a few mistakes on my part with the languages. Let me know if you see anything else.

@Golmote
Copy link
Contributor

Golmote commented Jul 12, 2015

Thanks. This looks quite good to me now! @LeaVerou what do you think?

@Golmote
Copy link
Contributor

Golmote commented Sep 16, 2015

Sorry for the delay, I'm merging this now.

Golmote added a commit that referenced this pull request Sep 16, 2015
adding system verilog language
@Golmote Golmote merged commit 44a11c2 into PrismJS:gh-pages Sep 16, 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.

None yet

2 participants