Skip to content

/ flutter

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

Jump to bottom

✒ Spell Check All .md Files Related to Flutter 💙 #61564

Merged
merged 62 commits into from Jul 23, 2020
Merged

✒ Spell Check All .md Files Related to Flutter 💙 #61564

merged 62 commits into from Jul 23, 2020
+258 −230

Conversation

@YazeedAlKhalaf
Copy link
Contributor

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 15, 2020 •
edited

Description

I have made spell correction for all .md files related and made Flutter. It is now more Solid. ALL grammar mistakes have been fixed 🐛. Although might some see it silly but I think it is one of the most important things to do from time to time.

Related Issues

This is voluntary. So no issues are related to this. I did it because I feel it is important.

Tests

This PR doesn't require a new test since it just changes .md files. Nothing else

Checklist

Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I updated/added relevant documentation (doc comments with ///). (Not Relevant to what I did)
  • All existing and new tests are passing. (Yes, I changed no real Flutter Code, just .md files)
  • The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
  • I am willing to follow-up on review comments in a timely manner.

Breaking Change

Did any tests fail when you ran them? Please read Handling breaking changes.

  • No, no existing tests failed, so this is not a breaking change.
  • Yes, this is a breaking change. If not, delete the remainder of this section.
@YazeedAlKhalaf
🐛 Fix Spelling Issues in Main README.md
134e976
@YazeedAlKhalaf
🐛 Fix spelling issues in dev README.md
d3187bc
@YazeedAlKhalaf
🐛 Fix spelling issues in complex_layout README.md
52c69aa
@YazeedAlKhalaf
🐛 Fix spelling issues in macrobenchmarks README.md
9ed4593
@YazeedAlKhalaf
🐛 Fix spelling issues in platform_views_layout README.md
09da77b
@YazeedAlKhalaf
🐛 Fix spelling issues in test_Apps/stocks README.md
96a00fb
@YazeedAlKhalaf
🐛 Fix spelling issues in bots README.md
be63ca7
@YazeedAlKhalaf
✒ Spell Check dev/ci README.md
bc67ad4
@YazeedAlKhalaf
✒ Spell Check dev/ci/docker_linux README.md
b6a09af
@YazeedAlKhalaf
✒ Spell Check dev/devicelab README.md
fb4444f
@YazeedAlKhalaf
✒ Spell Check dev/docs README.md
5488887
@YazeedAlKhalaf
✒ Spell Check dev/snippets README.md
9f1bb90
@YazeedAlKhalaf
✒ Spell Check dev/snippets/config/templates README.md
586db40
@YazeedAlKhalaf
✒ Spell Check dev/tools/gen_keycodes README.md
e19acd5
@YazeedAlKhalaf
✒ Spell Check dev/tools/vitool README.md
75c5f17
@YazeedAlKhalaf
✒ Spell Check examples/catalog README.md
8e5d081
@YazeedAlKhalaf
✒ Spell Check examples/flutter_view README.md
ea7ba9d
@YazeedAlKhalaf
✒ Spell Check examples/image_list README.md
38d7102
@YazeedAlKhalaf
✒ Spell Check examples/layers README.md
abd2007
@YazeedAlKhalaf
✒ Spell Check examples/platform_channel README.md
f564a85
@YazeedAlKhalaf
✒ Spell Check examples/platform_channel_swift README.md
b46a805
@YazeedAlKhalaf
✒ Spell Check examples/platform_view README.md
3631b19
@YazeedAlKhalaf
✒ Spell Check packages/_flutter_web_build_script README.md
6f27545
@YazeedAlKhalaf
✒ Spell Check packages/flutter_localizations README.md
f655f40
@YazeedAlKhalaf
✒ Spell Check packages/flutter_tools README.md
602a1a4
@YazeedAlKhalaf
✒ Spell Check CODE_OF_CONDUCT.md
f1fa601
@YazeedAlKhalaf
✒ Spell Check dev/integration_test/android_splash_screens/splash_Scre…
5b0f7b1 
…en_load_rotate README.md
@YazeedAlKhalaf
✒ Spell Check dev/integration_test/android_views README.md
c314f83
@YazeedAlKhalaf
✒ Spell Check dev/integration_tests/flutter_driver_screenshot_test RE…
76aa373 
…ADME.md
@YazeedAlKhalaf
✒ Spell Check dev/integration_tests/flutter_gallery README.md
a9a93a2
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 15, 2020

@christopherfujino Great!
I have made the changes except one, needs your answer.

No problem take your time
@YazeedAlKhalaf
❌ Remove trailing space
64302d9
@YazeedAlKhalaf
❌ Remove trailing space
5edbfdf
@christopherfujino
Copy link
Contributor

@christopherfujino christopherfujino commented Jul 16, 2020

@christopherfujino Great!
I have made the changes except one, needs your answer.

Sorry, which is the change you had a question on? I can't see it in the UI.
@christopherfujino
christopherfujino reviewed Jul 16, 2020
View changes
examples/flutter_view/README.md Outdated Show resolved Hide resolved
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 16, 2020

dev/integration_tests/gradle_deprecated_settings/README.md

This one @christopherfujino
@YazeedAlKhalaf
🐛 Fix small typo
2941a80
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 16, 2020 •
edited

Made All changes requested 😊
@christopherfujino
@YazeedAlKhalaf
✒ Fix Typo
382c47b
@christopherfujino
christopherfujino approved these changes Jul 17, 2020
View changes
Copy link
Contributor

@christopherfujino christopherfujino left a comment

Everything here LGTM (looks good to me)! Thanks again for all of your work here and sticking with my code review, this is an awesome change. Hope to see you around here again!
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 17, 2020

Thanks a lot @christopherfujino 😊🙏

Me too
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 18, 2020

Why is flutter-build failing?
@christopherfujino
@christopherfujino
Copy link
Contributor

@christopherfujino christopherfujino commented Jul 20, 2020

Why is flutter-build failing?
@christopherfujino

@YazeedAlKhalaf "Flutter build" means that the master branch of flutter is failing testing (this is unrelated to your change). If you want to see the exact reason, you can check our build dashboard: https://flutter-dashboard.appspot.com/#/build. I added the "waiting for tree to go green" label to your PR, which will usually signal a bot to merge the PR once the build dashboard is green. However, it looks like someone else's change caused a merge conflict on your PR (to the file dev/benchmarks/macrobenchmarks/README.md). This means that GitHub isn't sure how to cleanly merge your change with the one someone else already made, and manual intervention is needed. Usually for source code changes, I want to fix this in my text editor, so I will use git and merge upstream master into my feature branch, resolve the conflict in my editor, and then push the resolved version back up. However, since this is a documentation merge conflict, it's probably safe to use the web editor. If you see the "Resolve conflicts" button in GitHub, you can use that to resolve. Let me know if you need any other help!
@YazeedAlKhalaf
🐛 Fix small bug
c8e569b
@YazeedAlKhalaf
Merge branch 'master' of https://github.com/YazeedAlKhalaf/flutter
b6480f4
@YazeedAlKhalaf
Merge remote-tracking branch 'upstream/master'
4d4eaf4
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 20, 2020

Okay, thanks @christopherfujino

I have made the changes
@christopherfujino
christopherfujino reviewed Jul 20, 2020
View changes
dev/benchmarks/macrobenchmarks/README.md
@@ -4,6 +4,33 @@ Performance benchmarks use either flutter drive or the web benchmark harness.

## Mobile benchmarks

### Cull opacity benchmark

This comment has been minimized.

Sign in to view
@christopherfujino

christopherfujino Jul 20, 2020
Contributor

Your PR here is adding this text which I don't see on master, maybe this got deleted in a recent change and when merging back in master you accidentally restored it?

This comment has been minimized.

Sign in to view
@YazeedAlKhalaf

YazeedAlKhalaf Jul 21, 2020
Author Contributor

I guess but what I didi isn’t fetch all new things and see what was changed so I changed nothing but I will make sure

This comment has been minimized.

Sign in to view
@christopherfujino

christopherfujino Jul 21, 2020
Contributor

When there's a merge conflict you have to manually look through all the changes and decide which makes sense to keep. This change removed the this README text, so with your merge conflict you need to also remove it.

This comment has been minimized.

Sign in to view
@creativecreatorormaybenot

creativecreatorormaybenot Mar 2, 2021
Contributor

@christopherfujino You merged it like this, right?
@fluttergithubbot
Copy link
Contributor

@fluttergithubbot fluttergithubbot commented Jul 20, 2020

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite hostonly_devicelab_tests-3_last-linux has failed. Please fix the issues identified (or deflake) before re-applying this label.
@christopherfujino
Copy link
Contributor

@christopherfujino christopherfujino commented Jul 20, 2020

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite hostonly_devicelab_tests-3_last-linux has failed. Please fix the issues identified (or deflake) before re-applying this label.

This looks like a timeout (not related to this change), I re-ran it.
@jmagman jmagman merged commit d41b1fb into flutter:master Jul 23, 2020
60 checks passed
60 checks passed
@wip
WIP Ready for review
Details
@cirrus-ci
analyze-linux Task Summary
Details
@cirrus-ci
build_tests-0-linux Task Summary
Details
@cirrus-ci
build_tests-0-macos Task Summary
Details
@cirrus-ci
build_tests-0-windows Task Summary
Details
@cirrus-ci
build_tests-1_last-linux Task Summary
Details
@cirrus-ci
build_tests-1_last-macos Task Summary
Details
@cirrus-ci
build_tests-1_last-windows Task Summary
Details
@googlebot
cla/google All necessary CLAs are signed
@cirrus-ci
customer_testing-linux Task Summary
Details
@cirrus-ci
customer_testing-macos Task Summary
Details
@cirrus-ci
customer_testing-windows Task Summary
Details
@cirrus-ci
deploy_gallery-linux Task Summary
Details
@cirrus-ci
deploy_gallery-macos Task Summary
Details
@cirrus-ci
docs-linux Task Summary
Details
@cirrus-ci
firebase_test_lab_tests-0-linux Task Summary
Details
@cirrus-ci
firebase_test_lab_tests-1-linux Task Summary
Details
@cirrus-ci
firebase_test_lab_tests-2_last-linux Task Summary
Details
@fluttergithubbot
flutter-build
Details
@fluttergithubbot
flutter-gold All golden file tests have passed.
Details
@cirrus-ci
framework_tests-libraries-linux Task Summary
Details
@cirrus-ci
framework_tests-libraries-macos Task Summary
Details
@cirrus-ci
framework_tests-libraries-windows Task Summary
Details
@cirrus-ci
framework_tests-misc-linux Task Summary
Details
@cirrus-ci
framework_tests-misc-macos Task Summary
Details
@cirrus-ci
framework_tests-misc-windows Task Summary
Details
@cirrus-ci
framework_tests-widgets-linux Task Summary
Details
@cirrus-ci
framework_tests-widgets-macos Task Summary
Details
@cirrus-ci
framework_tests-widgets-windows Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-0-linux Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-0-macos Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-0-windows Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-1-linux Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-1-macos Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-1-windows Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-2-linux Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-2-macos Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-2-windows Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-3_last-linux Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-3_last-macos Task Summary
Details
@cirrus-ci
hostonly_devicelab_tests-3_last-windows Task Summary
Details
@cirrus-ci
tool_tests-commands-linux Task Summary
Details
@cirrus-ci
tool_tests-commands-macos Task Summary
Details
@cirrus-ci
tool_tests-commands-windows Task Summary
Details
@cirrus-ci
tool_tests-general-linux Task Summary
Details
@cirrus-ci
tool_tests-general-macos Task Summary
Details
@cirrus-ci
tool_tests-general-windows Task Summary
Details
@cirrus-ci
tool_tests-integration-linux Task Summary
Details
@cirrus-ci
tool_tests-integration-macos Task Summary
Details
@cirrus-ci
tool_tests-integration-windows Task Summary
Details
@cirrus-ci
web_integration_tests Task Summary
Details
@cirrus-ci
web_smoke_test Task Summary
Details
@cirrus-ci
web_tests-0-linux Task Summary
Details
@cirrus-ci
web_tests-1-linux Task Summary
Details
@cirrus-ci
web_tests-2-linux Task Summary
Details
@cirrus-ci
web_tests-3-linux Task Summary
Details
@cirrus-ci
web_tests-4-linux Task Summary
Details
@cirrus-ci
web_tests-5-linux Task Summary
Details
@cirrus-ci
web_tests-6-linux Task Summary
Details
@cirrus-ci
web_tests-7_last-linux Task Summary
Details
@jonahwilliams
Copy link
Member

@jonahwilliams jonahwilliams commented Jul 23, 2020

This is a great change, thank you so much @YazeedAlKhalaf !
@YazeedAlKhalaf
Copy link
Contributor Author

@YazeedAlKhalaf YazeedAlKhalaf commented Jul 23, 2020

This is a great change, thank you so much @YazeedAlKhalaf !

@jonahwilliams You’re welcome 🚀
Pragya007 added a commit to Pragya007/flutter that referenced this pull request Aug 11, 2020
@YazeedAlKhalaf @Pragya007
✒ Spell Check All .md Files Related to Flutter 💙 (flutter#61564)
41e7ec9 
* 🐛 Fix Spelling Issues in Main README.md

* 🐛 Fix spelling issues in dev README.md

* 🐛 Fix spelling issues in complex_layout README.md

* 🐛 Fix spelling issues in macrobenchmarks README.md

* 🐛 Fix spelling issues in platform_views_layout README.md

* 🐛 Fix spelling issues in test_Apps/stocks README.md

* 🐛 Fix spelling issues in bots README.md

* ✒ Spell Check dev/ci README.md

* ✒ Spell Check dev/ci/docker_linux README.md

* ✒ Spell Check dev/devicelab README.md

* ✒ Spell Check dev/docs README.md

* ✒ Spell Check dev/snippets README.md

* ✒ Spell Check dev/snippets/config/templates README.md

* ✒ Spell Check dev/tools/gen_keycodes README.md

* ✒ Spell Check dev/tools/vitool README.md

* ✒ Spell Check examples/catalog README.md

* ✒ Spell Check examples/flutter_view README.md

* ✒ Spell Check examples/image_list README.md

* ✒ Spell Check examples/layers README.md

* ✒ Spell Check examples/platform_channel README.md

* ✒ Spell Check examples/platform_channel_swift README.md

* ✒ Spell Check examples/platform_view README.md

* ✒ Spell Check packages/_flutter_web_build_script README.md

* ✒ Spell Check packages/flutter_localizations README.md

* ✒ Spell Check packages/flutter_tools README.md

* ✒ Spell Check CODE_OF_CONDUCT.md

* ✒ Spell Check dev/integration_test/android_splash_screens/splash_Screen_load_rotate README.md

* ✒ Spell Check dev/integration_test/android_views README.md

* ✒ Spell Check dev/integration_tests/flutter_driver_screenshot_test README.md

* ✒ Spell Check dev/integration_tests/flutter_gallery README.md

* ✒ Spell Check dev/integration_tests/gradle_deprecated_settings README.md

* ✒ Spell Check dev/integration_tests/ios_add2app_life_cycle README.md

* ✒ Spell Check dev/integration_tests/ios_host_app README.md

* ✒ Spell Check dev/integration_tests/ios_platform_view_tests README.md

* ✒ Spell Check dev/automated_tests/flutter_test README.md

* ✒ Spell Check .github/PULL_REQUEST_TEMPLATE.md

* ✒ Spell Check .hithub/ISSUE_TEMPLATE/ACTIVATION.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/BUG.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/feature_request.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/performance_others.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/performance_speed.md

* ✒ Spell Check packages/flutter_tools/doc/daemon.md

* ✒ Spell Check packages/flutter_tools/fuchsia_enrtypoint_shim/README.md

* ✒ Minimize line to 80 columns

* ✒ Minimize line to 80 columns

* ✒ Fix Typo

* ✒ Chnaged numbers to 1 for testing purposes

* ✒ Changed numbers to 1 for testing purposes

* ❌ Remove 'a' which is a typo

* ✒ Change a sentence to be better

* ✒ Remove 'a' which is a typo

* ✒ Fix small issue

* ✒ Fix small typo

* ✒ Fix some typos

* ❌ Remove trailing space

* ❌ Remove trailing space

* 🐛 Fix small typo

* ✒ Fix Typo

* 🐛 Fix small bug
mingwandroid added a commit to mingwandroid/flutter that referenced this pull request Sep 6, 2020
@YazeedAlKhalaf @mingwandroid
✒ Spell Check All .md Files Related to Flutter 💙 (flutter#61564)
c774e5c 
* 🐛 Fix Spelling Issues in Main README.md

* 🐛 Fix spelling issues in dev README.md

* 🐛 Fix spelling issues in complex_layout README.md

* 🐛 Fix spelling issues in macrobenchmarks README.md

* 🐛 Fix spelling issues in platform_views_layout README.md

* 🐛 Fix spelling issues in test_Apps/stocks README.md

* 🐛 Fix spelling issues in bots README.md

* ✒ Spell Check dev/ci README.md

* ✒ Spell Check dev/ci/docker_linux README.md

* ✒ Spell Check dev/devicelab README.md

* ✒ Spell Check dev/docs README.md

* ✒ Spell Check dev/snippets README.md

* ✒ Spell Check dev/snippets/config/templates README.md

* ✒ Spell Check dev/tools/gen_keycodes README.md

* ✒ Spell Check dev/tools/vitool README.md

* ✒ Spell Check examples/catalog README.md

* ✒ Spell Check examples/flutter_view README.md

* ✒ Spell Check examples/image_list README.md

* ✒ Spell Check examples/layers README.md

* ✒ Spell Check examples/platform_channel README.md

* ✒ Spell Check examples/platform_channel_swift README.md

* ✒ Spell Check examples/platform_view README.md

* ✒ Spell Check packages/_flutter_web_build_script README.md

* ✒ Spell Check packages/flutter_localizations README.md

* ✒ Spell Check packages/flutter_tools README.md

* ✒ Spell Check CODE_OF_CONDUCT.md

* ✒ Spell Check dev/integration_test/android_splash_screens/splash_Screen_load_rotate README.md

* ✒ Spell Check dev/integration_test/android_views README.md

* ✒ Spell Check dev/integration_tests/flutter_driver_screenshot_test README.md

* ✒ Spell Check dev/integration_tests/flutter_gallery README.md

* ✒ Spell Check dev/integration_tests/gradle_deprecated_settings README.md

* ✒ Spell Check dev/integration_tests/ios_add2app_life_cycle README.md

* ✒ Spell Check dev/integration_tests/ios_host_app README.md

* ✒ Spell Check dev/integration_tests/ios_platform_view_tests README.md

* ✒ Spell Check dev/automated_tests/flutter_test README.md

* ✒ Spell Check .github/PULL_REQUEST_TEMPLATE.md

* ✒ Spell Check .hithub/ISSUE_TEMPLATE/ACTIVATION.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/BUG.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/feature_request.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/performance_others.md

* ✒ Spell Check .github/ISSUE_TEMPLATE/performance_speed.md

* ✒ Spell Check packages/flutter_tools/doc/daemon.md

* ✒ Spell Check packages/flutter_tools/fuchsia_enrtypoint_shim/README.md

* ✒ Minimize line to 80 columns

* ✒ Minimize line to 80 columns

* ✒ Fix Typo

* ✒ Chnaged numbers to 1 for testing purposes

* ✒ Changed numbers to 1 for testing purposes

* ❌ Remove 'a' which is a typo

* ✒ Change a sentence to be better

* ✒ Remove 'a' which is a typo

* ✒ Fix small issue

* ✒ Fix small typo

* ✒ Fix some typos

* ❌ Remove trailing space

* ❌ Remove trailing space

* 🐛 Fix small typo

* ✒ Fix Typo

* 🐛 Fix small bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christopherfujino christopherfujino

@creativecreatorormaybenot creativecreatorormaybenot

Assignees
No one assigned
Labels
a: internationalization cla: yes d: examples team tool waiting for tree to go green
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants
@YazeedAlKhalaf @googlebot @christopherfujino @fluttergithubbot @jonahwilliams @creativecreatorormaybenot @jmagman