docs: spelling corrections#5349
docs: spelling corrections#5349jsumners merged 5 commits intofastify:mainfrom 10xLaCroixDrinker:docs/spelling
Conversation
Fdawgs
left a comment
There was a problem hiding this comment.
Will review properly next week.
FYI, Fastify uses American/Simple English over British/Traditional English, so "Behavior" is correct.
jsumners
left a comment
There was a problem hiding this comment.
Other than my comments (and subsequent lines that are the same but I left uncommented), this looks good. Please resolve the issues highlighted.
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'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.) |
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 |
|
@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 |
|
let's do it |
Co-authored-by: Frazer Smith <frazer.dev@outlook.com> Signed-off-by: Jamie King <hello@jamieking.me>
|
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. |
Just fixing some spelling errors through docs, comments & tests
Checklist
npm run testandnpm run benchmarkand the Code of conduct