Skip to content

Spell check of Flutter docs #45200

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
Merged
gspencergoog merged 3 commits into from
Nov 22, 2019
Merged

Spell check of Flutter docs #45200

gspencergoog merged 3 commits into from
Nov 22, 2019

Conversation

@gspencergoog
Copy link
Contributor

@gspencergoog gspencergoog commented Nov 19, 2019

No code changes, just comments: I spell-checked all the comments in the repo.

@fluttergithubbot fluttergithubbot added a: internationalization Supporting other languages or locales. (aka i18n) a: tests "flutter test", flutter_test, or one of our tests d: examples Sample code and demos f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Nov 19, 2019
@gspencergoog gspencergoog added d: api docs Issues with https://api.flutter.dev/ documentation and removed a: internationalization Supporting other languages or locales. (aka i18n) a: tests "flutter test", flutter_test, or one of our tests cla: yes d: examples Sample code and demos f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels. labels Nov 19, 2019
natebosch
natebosch reviewed Nov 19, 2019
View reviewed changes
Copy link
Contributor

@natebosch natebosch left a comment

Choose a reason for hiding this comment

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

I think this spell checker may be too zealous on compound words...

@gspencergoog
Copy link
Contributor Author

gspencergoog commented Nov 20, 2019 •
edited
Loading

Hmm. I much prefer all of the hyphenated ones, but maybe that just makes me uncommon (un-common? :-) ). The non-hyphenated versions all look like made up words to me.

I reverted all the hyphenations I added, and all other hyphenations of these words in all the comments, so that we're at least consistent.

I kept plain text because it feels more descriptive and less a term of art (and it's nearly as common).

@gspencergoog gspencergoog changed the title Spell check of flutter docs Spell check of Flutter docs Nov 20, 2019
@fluttergithubbot fluttergithubbot added a: accessibility Accessibility, e.g. VoiceOver or TalkBack. (aka a11y) a: internationalization Supporting other languages or locales. (aka i18n) a: tests "flutter test", flutter_test, or one of our tests d: examples Sample code and demos f: cupertino flutter/packages/flutter/cupertino repository labels Nov 20, 2019
@gspencergoog gspencergoog removed a: tests "flutter test", flutter_test, or one of our tests d: examples Sample code and demos f: cupertino flutter/packages/flutter/cupertino repository f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels. tool Affects the "flutter" command-line tool. See also t: labels. labels Nov 20, 2019
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
Hixie
Hixie reviewed Nov 20, 2019
View reviewed changes
@Hixie
Copy link
Contributor

Hixie commented Nov 20, 2019

LGTM

@gspencergoog gspencergoog force-pushed the spell_check branch 2 times, most recently from 67e38ee to 773557a Compare November 21, 2019 16:44
darielopper
darielopper reviewed Nov 22, 2019
View reviewed changes
darielopper
darielopper reviewed Nov 22, 2019
View reviewed changes
darielopper
darielopper suggested changes Nov 22, 2019
View reviewed changes
@gspencergoog gspencergoog mentioned this pull request Nov 22, 2019
Merged
9 tasks
@gspencergoog gspencergoog merged commit a60bf8e into flutter:master Nov 22, 2019
@gspencergoog gspencergoog deleted the spell_check branch January 16, 2020 22:12
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@Hixie Hixie Hixie left review comments

@natebosch natebosch natebosch left review comments

+2 more reviewers

@jonahwilliams jonahwilliams jonahwilliams left review comments

@darielopper darielopper darielopper requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt. d: api docs Issues with https://api.flutter.dev/

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@gspencergoog @Hixie @natebosch @jonahwilliams @darielopper @googlebot @fluttergithubbot