Skip to content
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

Fix a few minor typos #22722

Closed
wants to merge 3 commits into from
Closed

Fix a few minor typos #22722

wants to merge 3 commits into from

Conversation

KonstHardy
Copy link
Contributor

@KonstHardy KonstHardy commented Nov 8, 2021

Hello,
I've found some minor typos in the documentation, solved with this PR.

Fix typo:
we we -> we
typechecking -> type checking
typecheck -> type check
reentrancy -> re-entrance

Hello,
I removed the redundant word in the documentation.

Fix typo:
we we -> we
@sizebot
Copy link

sizebot commented Nov 8, 2021

Comparing: 327d5c4...cbd8ad7

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 129.55 kB 129.55 kB = 41.43 kB 41.43 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 134.52 kB 134.52 kB = 42.90 kB 42.89 kB
facebook-www/ReactDOM-prod.classic.js = 423.36 kB 423.36 kB = 78.05 kB 78.05 kB
facebook-www/ReactDOM-prod.modern.js = 411.92 kB 411.92 kB = 76.30 kB 76.30 kB
facebook-www/ReactDOMForked-prod.classic.js = 423.36 kB 423.36 kB = 78.06 kB 78.06 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against cbd8ad7

@KonstHardy KonstHardy changed the title Fix minor typo Fix a few minor typos Nov 11, 2021
@bvaughn
Copy link
Contributor

bvaughn commented Nov 15, 2021

Thanks for this PR 🙇🏼 but "reentrancy" is a valid word, and "type check" vs "typecheck" seems kind of arbitrary. I'm going to close. :)

@bvaughn bvaughn closed this Nov 15, 2021
@KonstHardy KonstHardy deleted the fix-typo branch November 15, 2021 16:11
@KonstHardy
Copy link
Contributor Author

Thanks for this PR 🙇🏼 but "reentrancy" is a valid word, and "type check" vs "typecheck" seems kind of arbitrary. I'm going to close. :)

No problem, thanks for looking at my PR :)

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.

None yet

4 participants