Skip to content

Correct some spelling mistakes and reword some sentences #25

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

Merged
merged 1 commit into from
Jul 10, 2020

Conversation

TomCaserta
Copy link
Contributor

@TomCaserta TomCaserta commented Jul 7, 2020

Just a few small changes to the README, I may have gone a little too far with some of the rewording in places where it wasn't entirely necessary but hopefully this improves the quality of the readme a little.

If you'd like me to separate the spelling changes from the rewording so you can accept it separately I can do that.

I'm up for writing some more documentation in the future once I've put in some more time with the project.


[QuickJS](https://bellard.org/quickjs/) is used as the ECMAScript engine.
> QuickJS is a small and embeddable Javascript engine. It supports the ES2020 specification including modules, asynchronous generators and proxies.
>
> It optionally supports mathematical extensions such as big integers (BigInt), big floating point numbers (BigFloat) and operator overloading.

It is also possible to replace the ECMAScript engine like V8 or SpiderMonkey but not in the near plan.
In the future it may also be possible to replace the ECMAScript engine with other engines such as V8 or SpiderMonkey.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We can drop this before a better JS engine appeare.

@Geequlim Geequlim merged commit 557404b into godotjs:master Jul 10, 2020
@Geequlim
Copy link
Collaborator

Thanks

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