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

Update spelling in docs #73

Merged
merged 2 commits into from
Jan 24, 2022
Merged

Update spelling in docs #73

merged 2 commits into from
Jan 24, 2022

Conversation

msinton
Copy link
Contributor

@msinton msinton commented Jan 24, 2022

While "synthetize" is a correct spelling it appears to be an uncommon form of the word. (I initially thought incorrect)

IMO "synthesize CLI tools" seems a little odd, would consider if this is really what is meant.

If update agreed, also needed here: modules/docs/src/03-protocols/02-simple-rest-json/01-overview.md

While "synthetize" is a correct spelling it appears to be an uncommon form of the word. (I initially thought incorrect)

IMO "synthesize CLI tools" seems a little odd, would consider if this is really what is meant.
@CLAassistant
Copy link

CLAassistant commented Jan 24, 2022

CLA assistant check
All committers have signed the CLA.

@Baccata
Copy link
Contributor

Baccata commented Jan 24, 2022

synthesize CLI tools

I suppose "synthesise" could be replaced by "derive"

@kubukoz
Copy link
Member

kubukoz commented Jan 24, 2022

synthesize/synthesise is an American/British thing, isn't it?

@Baccata
Copy link
Contributor

Baccata commented Jan 24, 2022

@kubukoz after checking, it appears so : https://dictionary.cambridge.org/dictionary/english/synthesize

Regardless, the term could be changed to "derive", or "produce"

@msinton
Copy link
Contributor Author

msinton commented Jan 24, 2022

@kubukoz @Baccata I've updated all usages to "derive" which seemed like the best option to me

@kubukoz kubukoz changed the title spelling Update spelling in docs Jan 24, 2022
@kubukoz
Copy link
Member

kubukoz commented Jan 24, 2022

looks good, would you mind updating the other files you mentioned too?

@msinton
Copy link
Contributor Author

msinton commented Jan 24, 2022

looks good, would you mind updating the other files you mentioned too?

already done 👍

@Baccata
Copy link
Contributor

Baccata commented Jan 24, 2022

Thanks !

@Baccata Baccata merged commit d1fa1f4 into disneystreaming:main Jan 24, 2022
@msinton msinton deleted the patch-1 branch January 24, 2022 16:14
@kubukoz
Copy link
Member

kubukoz commented Jan 24, 2022

looks good, would you mind updating the other files you mentioned too?

already done 👍

ah, sorry, the file path in the original post made me think there was more to it on a quick skim :D

Baccata pushed a commit that referenced this pull request May 10, 2022
 Update wording, derive over synthetize
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.

None yet

4 participants