Skip to content

docs: spelling corrections#5349

Merged
jsumners merged 5 commits intofastify:mainfrom
10xLaCroixDrinker:docs/spelling
Mar 11, 2024
Merged

docs: spelling corrections#5349
jsumners merged 5 commits intofastify:mainfrom
10xLaCroixDrinker:docs/spelling

Conversation

@10xLaCroixDrinker
Copy link
Copy Markdown
Contributor

Just fixing some spelling errors through docs, comments & tests

Checklist

@metcoder95 metcoder95 requested a review from Fdawgs March 8, 2024 08:55
Copy link
Copy Markdown
Member

@Fdawgs Fdawgs left a comment

Choose a reason for hiding this comment

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

Will review properly next week.
FYI, Fastify uses American/Simple English over British/Traditional English, so "Behavior" is correct.

Copy link
Copy Markdown
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

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

Other than my comments (and subsequent lines that are the same but I left uncommented), this looks good. Please resolve the issues highlighted.

Copy link
Copy Markdown
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@10xLaCroixDrinker
Copy link
Copy Markdown
Contributor Author

FYI, Fastify uses American/Simple English over British/Traditional English

Should https://fastify.dev/organisations/ be https://fastify.dev/organizations/ then? Is that a PR you'd be open to (with a redirect of course)?

@jsumners
Copy link
Copy Markdown
Member

jsumners commented Mar 8, 2024

I'll leave that up to the @fastify/website team. But I don't think we need to be so pedantic as to break the website.

(FYI: I'm American, having never left the country in my 4+ decades [record to be broke in April 🎉], and I routinely write British variants of words before spellcheck kicks in. I'm not a stickler for the variants. We just have a style guide and should adhere to it as much as makes sense for consistency.)

Copy link
Copy Markdown
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@gurgunday
Copy link
Copy Markdown
Member

FYI, Fastify uses American/Simple English over British/Traditional English

Should https://fastify.dev/organisations/ be https://fastify.dev/organizations/ then? Is that a PR you'd be open to (with a redirect of course)?

I think for consistency sake it should be "organizations", is it hard to add a redirect to "organisations"? Not familiar with what framework the website uses

@10xLaCroixDrinker
Copy link
Copy Markdown
Contributor Author

@gurgunday the site is built with docusaurus and already uses a plugin for a few other redirects, so it would just take a couple additional lines to that plugin config

@mcollina
Copy link
Copy Markdown
Member

mcollina commented Mar 9, 2024

let's do it

10xLaCroixDrinker and others added 2 commits March 10, 2024 15:47
Co-authored-by: Frazer Smith <frazer.dev@outlook.com>
Signed-off-by: Jamie King <hello@jamieking.me>
@jsumners jsumners merged commit c957be2 into fastify:main Mar 11, 2024
@10xLaCroixDrinker 10xLaCroixDrinker deleted the docs/spelling branch March 11, 2024 18:18
@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants