-
Notifications
You must be signed in to change notification settings - Fork 3.9k
BigInt Type added in primitive types #1620
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
Conversation
BigInt is currently in stage 4 of approving and is just left to be shipped to Safari rest its applicable everywhere I think we should add it. |
It's not in https://tc39.es/ecma262/ yet though |
It's there in it. That is why I posted it. It's already implemented in Chrome and Mozilla only Safari implementation is left. That too is going on. |
I shouldn't have accepted this PR. It's terrible. It ignores all the flow of the tutorial and just sticks BigInt where it shouldn't be. |
and uses |
Please tell me how can I improve I didn't know in which firm you want this. |
I will rewrite that. Just please look at the tutorial in future PRs, so that you don't use unexplained operators in your code. Our purpose is to explain things well, not just list them as the manual does. |
I've just started contribution to open source please forgive me this time I'll learn from the way you articulate your PR and terms I'm really sorry for this time. Thank you for for correcting my PR |
thank you for your efforts, surely you can do better |
The new Bigint type was not added to the primitive types I added it to the primitive type please tell me where ever it needs to be added. So that the whole documentation is consistent everywhere.