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 Apex to supported languages #3546

Merged
merged 9 commits into from
Jun 15, 2022
Merged

Add Apex to supported languages #3546

merged 9 commits into from
Jun 15, 2022

Conversation

dschach
Copy link
Contributor

@dschach dschach commented May 31, 2022

Add Apex (Salesforce) language

Changes

Include Apex in table

Checklist

  • Added markup tests, or they don't apply here because... All tests are in the repo
  • Updated the changelog at CHANGES.md

@dschach dschach marked this pull request as ready for review May 31, 2022 19:30
Version 1.2.5 to 1.2.6
@dschach
Copy link
Contributor Author

dschach commented Jun 1, 2022

Apex is now also on NPM (and all the CDN services automatically).
https://www.npmjs.com/package/highlightjs-apex

CHANGES.md Outdated Show resolved Hide resolved
package-lock.json Outdated Show resolved Hide resolved
SUPPORTED_LANGUAGES.md Outdated Show resolved Hide resolved
Planning to remove lightning from syntax alias because may use it for Lightning Web Component javascript syntax in the future.
@dschach dschach requested a review from joshgoebel June 12, 2022 16:21
@joshgoebel
Copy link
Member

Please do not reformat the entire markdown file, just add the single line you need (so the diff is clear on what's changed), it's doesn't matter whether the brackets are perfect or not in the ASCII form just so long as it render fine on GitHub...

@dschach
Copy link
Contributor Author

dschach commented Jun 15, 2022

Fixed. Ready for review.

Copy link
Member

@joshgoebel joshgoebel left a comment

Choose a reason for hiding this comment

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

Thanks so much! Perfect!

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.

2 participants