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.

Sign up for GitHub

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

Jump to bottom

fix typos #36908

Merged
merged 1 commit into from
Oct 4, 2016
Merged

fix typos #36908

merged 1 commit into from
Oct 4, 2016

Conversation

GuillaumeGomez
Copy link
Member

@apasel422
Copy link
Contributor

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Oct 2, 2016

📌 Commit a638580 has been approved by apasel422

@frewsxcv
Copy link
Member

frewsxcv commented Oct 3, 2016

For what it's worth:

  • British English: initialised
  • American English: initialized

@GuillaumeGomez
Copy link
Member Author

Why are there so much differences? >.<

@nagisa
Copy link
Member

nagisa commented Oct 3, 2016 •
edited
Loading

Pretty much every case of -zed is American English specific by now. All other Englishes using -sed. (similarly for -ize and -ise). That being said our documentation is predominantly American English (however it pains me being this way), so this change is “correct”.

EDIT: if there’s some interest, the Oxford dictionary has a nice preface on it.

@Manishearth Manishearth mentioned this pull request Oct 4, 2016
Merged
Manishearth added a commit to Manishearth/rust that referenced this pull request Oct 4, 2016
@Manishearth
Rollup merge of rust-lang#36908 - GuillaumeGomez:default_doc, r=apase…
268521f 
…l422

fix typos

r? @steveklabnik
bors added a commit that referenced this pull request Oct 4, 2016
@bors
Auto merge of #36953 - Manishearth:rollup, r=Manishearth
4a9af01 
Rollup of 12 pull requests

- Successful merges: #36798, #36878, #36902, #36903, #36908, #36916, #36917, #36921, #36928, #36938, #36941, #36951
- Failed merges:
@bors bors merged commit a638580 into rust-lang:master Oct 4, 2016
@GuillaumeGomez GuillaumeGomez deleted the default_doc branch October 4, 2016 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@steveklabnik steveklabnik

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@GuillaumeGomez @apasel422 @bors @frewsxcv @nagisa @steveklabnik