BigInt is rendered as error

[hendrysadrak]

Prerequisites

• Put an X between the brackets on this line if you have done all of the following:
  • Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
  • Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
  • Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
  • Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
  • Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

Here is an example of the error:

const veryLargeInt = 100n ** 200n;

This without bigint:

const veryLargeInt = 100 ** 200;

[thomasjo]

Thanks for the report!

Because BigInt is still a draft proposal, I'm going to close this issue. Once the proposal gets accepted, feel free to leave a comment requesting the issue be re-opened, or open a new issue.

[NN---]

BigInt has already reached Stage3 which means it will be likely to be accepted.

[michaelficarra]

@thomasjo BigInts still aren't supported by this project, though they are officially part of the language now. See https://tc39.es/ecma262/#sec-bigint-objects. Please re-open.

[thomasjo]

@michaelficarra I'm no longer involved in the Atom project, so I cannot really champion for this issue getting resolved. I'll reopen it however, in hopes that someone might submit a PR to resolve it.

[lkashef]

There is a PR that seems to fix this issue #588

[lkashef]

This issue is no longer reproducible, it seems that tree-sitter/tree-sitter-javascript#93 fixed the issue, if you are able to reproduce, please feel free to open a new issue with the proper template and reproduction steps.

[michaelficarra]

GitHub linguist points to this repository as the syntax highlighting provider for JavaScript: https://github.com/github/linguist/blob/master/vendor/README.md

They currently reference d668216, or version 0.132.0, as a dependency. @lkashef Are you saying that support for BigInt was added between then and the latest release, 0.133.1, and if they bump the version, they will get that support?

edit: It looks like the tree-sitter dependency did not change between these versions: see https://github.com/atom/language-javascript/blob/v0.132.0/package.json and https://github.com/atom/language-javascript/blob/v0.133.1/package.json.

edit again: It seems like GitHub linguist uses the grammars/javascript.cson file in this repo, which still requires bigint support.