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 ABAP language #636

Merged
merged 6 commits into from
Sep 11, 2015
Merged

Add ABAP language #636

merged 6 commits into from
Sep 11, 2015

Conversation

dellagustin
Copy link
Contributor

Based on the work of @rmtiwari (https://github.com/rmtiwari/ABAPSyntaxHighlighter).
Supports:

  • line comments
  • end of line comments
  • numbers
  • operators
  • strings
  • string templates
  • punctuation
  • token operators
  • huge (probably complete) list of keywords

@@ -0,0 +1,50 @@
Prism.languages.abap= {
Copy link
Contributor

Choose a reason for hiding this comment

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

May you add a space before the =?

@Golmote
Copy link
Contributor

Golmote commented Jul 10, 2015

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

@dellagustin
Copy link
Contributor Author

Thanks for the comments, I did the changes.

lookbehind: true
},
'string_operator' : {
pattern: /(\s)(&{1,2})(?=\s)/,
Copy link
Contributor

Choose a reason for hiding this comment

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

The parentheses aren't needed around &{1,2}. Also, it would be shorter written as &&?. (BTW this also apply to \*{1,2} in the operator regexp above.)

@Golmote
Copy link
Contributor

Golmote commented Jul 11, 2015

This looks good to me. @LeaVerou, can you take a look, too?

@Golmote
Copy link
Contributor

Golmote commented Sep 11, 2015

Sorry for the delay, I'm merging this.

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

2 participants