Skip to content

Fix spelling in introduction.mdx #191

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 2 commits into from
Jan 25, 2021
Merged

Fix spelling in introduction.mdx #191

merged 2 commits into from
Jan 25, 2021

Conversation

svenheden
Copy link
Contributor

No description provided.

@ryyppy
Copy link
Member

ryyppy commented Jan 14, 2021

maybe i am wrong, but i am pretty sure that our current version is spelled correctly 😄

@kevanstannard
Copy link
Contributor

Hi @ryyppy I believe it's a grammar correction, might be worth another quick check?

@fhammerschmidt
Copy link
Member

Shouldn't it say "We have one of the fastest compilers & build systems"? Because "we" is probably the ReScript team or community. The compiler did not write the docs. 😉

@@ -20,7 +20,7 @@ We respect TypeScript very much and think that it's a positive force in the Java
- Is deliberately curated to be a simple subset most folks will have an easier time to use.
- Has **no** pitfalls, aka the type system is "sound" (the types will always be correct). E.g. If a type isn't marked as nullable, its value will never lie and let through some `undefined` value silently. **ReScript code has no null/undefined errors**.
- Is the same for everyone. No knobs, no bikeshedding opportunity.
- Runs extremely fast precisely thanks to its simplicity and curation. We're one of the fastest compiler & build system for JavaScript development.
- Runs extremely fast precisely thanks to its simplicity and curation. We're one of the fastest compiler & build systems for JavaScript development.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is minor, but with context;

Current:

In contrast, ReScript's type system:

  • Runs extremely fast precisely thanks to its simplicity and curation. We're one of the fastest compiler & build system for JavaScript development.

Suggestion:

In contrast, ReScript's type system:

  • Runs extremely fast precisely thanks to its simplicity and curation. It's one of the fastest compiler & build systems for JavaScript development.

@IwanKaramazow
Copy link
Collaborator

I'm not a native English speaker, but the grammar correction sounds ok? If you put something simple in the sentence: "we are one of the best piano players in the world". I would naturally assume that piano players is plural, so systems would also be plural?

@ryyppy
Copy link
Member

ryyppy commented Jan 16, 2021

It feels weird and asymmetric to me to have "compiler" in singular, but "build systems" in plural?

Maybe it would also help to call it "compiler & build system toolchains", if plural is the grammatically correct form?

I like the "It's one of..." version better than "We're the fastest..." btw

@svenheden
Copy link
Contributor Author

svenheden commented Jan 19, 2021

Good suggestions! Updated now

@ryyppy ryyppy merged commit aaa193c into rescript-lang:master Jan 25, 2021
@ryyppy
Copy link
Member

ryyppy commented Jan 25, 2021

thanks!

@svenheden svenheden deleted the patch-2 branch January 27, 2021 12:42
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.

5 participants