Skip to content

JS: Added support for numeric separators#1895

Merged
RunDevelopment merged 2 commits into
PrismJS:masterfrom
RunDevelopment:js-numeric-separator
May 18, 2019
Merged

JS: Added support for numeric separators#1895
RunDevelopment merged 2 commits into
PrismJS:masterfrom
RunDevelopment:js-numeric-separator

Conversation

@RunDevelopment

Copy link
Copy Markdown
Member

This fixes #1893.

The number pattern is quite a bit longer because of this change, so any ideas on how we could simplify this are more than welcome.


Spec changes.

Comment thread tests/languages/javascript/number_feature.test
@RunDevelopment
RunDevelopment merged commit 6068bf1 into PrismJS:master May 18, 2019
@RunDevelopment
RunDevelopment deleted the js-numeric-separator branch May 18, 2019 15:57
@RunDevelopment

Copy link
Copy Markdown
Member Author

@mathiasbynens Thank you for reviewing!

@mathiasbynens

Copy link
Copy Markdown
Contributor

@RunDevelopment Thanks for the quick patch!

There’s no rush, but what’s the general process for releasing a new prism-js version on npm? Is there a fixed release schedule or something like that?

@RunDevelopment

Copy link
Copy Markdown
Member Author

There is no fixed release schedule. A new version usually gets releases once enough changes have been made.
Looking at the committed changes since 1.16 and the pending PRs, I would say that we will release 1.17 soon (maybe in a few weeks or so).

mathiasbynens added a commit to v8/v8.dev that referenced this pull request May 19, 2019
This applies @RunDevelopment’s patch from PrismJS/prism#1895. It can be removed once a new prism-js release is published to npm.

Closes #182.
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.

JavaScript numeric separators

2 participants