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

.ino is main file extension for arduino files #2990

Merged
merged 4 commits into from
Oct 4, 2021

Conversation

Outreach-ChemEng-UofU
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Jul 9, 2021

JS File Size Changes (gzipped)

A total of 3 files have changed, with a combined diff of +26 B (+0.4%).

file master pull size diff % diff
components/prism-arduino.min.js 1.91 KB 1.92 KB +11 B +0.6%
plugins/autoloader/prism-autoloader.min.js 2.29 KB 2.29 KB +7 B +0.3%
plugins/show-language/prism-show-language.min.js 2.48 KB 2.49 KB +8 B +0.3%

Generated by 🚫 dangerJS against 4dc5f92

@joshgoebel
Copy link

Interesting, does Prism do aliases like this?

@Outreach-ChemEng-UofU
Copy link
Contributor Author

Outreach-ChemEng-UofU commented Jul 9, 2021

Interesting, does Prism do aliases like this?

I looked at the python language and that's how they handled py and python

Tested it with FileHighlight to remote .ino file and the proper code highlighting started working.

@joshgoebel
Copy link

Ruby also. Prism.languages.rb = Prism.languages.ruby;

Perhaps you are right. :) Just wasn't aware of this.

Copy link
Member

@RunDevelopment RunDevelopment left a comment

Choose a reason for hiding this comment

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

Thank you for the PR @Outreach-ChemEng-UofU!

To add an alias, you also have to register it in components.json using the "alias" property. You also have to rebuild Prism by running the command npm ci && npm run build.


@joshgoebel Yes, this is how Prism currently does aliases.

@RunDevelopment RunDevelopment merged commit 5b7ce5e into PrismJS:master Oct 4, 2021
@RunDevelopment
Copy link
Member

Thank you for contributing @Outreach-ChemEng-UofU!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants