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 README.md #155206

Merged
merged 3 commits into from
Sep 20, 2024
Merged

Update README.md #155206

merged 3 commits into from
Sep 20, 2024

Conversation

ArtificialZeng
Copy link
Contributor

@ArtificialZeng ArtificialZeng commented Sep 14, 2024

This PR changes

libraries like Skia (that underpins Chrome and Android)

to:

libraries like Skia (which underpins Chrome and Android)

This comment was marked as resolved.

Copy link
Member

@nate-thegrate nate-thegrate left a comment

Choose a reason for hiding this comment

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

If you're good with reverting the "the", I'd be happy to approve a 1-word change!

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@nate-thegrate
Copy link
Member

Looks like you'll also need to sign the CLA, per the comment above.

@ArtificialZeng
Copy link
Contributor Author

If you're good with reverting the "the", I'd be happy to approve a 1-word change!

thank you very much.

Copy link
Contributor Author

@ArtificialZeng ArtificialZeng left a comment

Choose a reason for hiding this comment

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

lgtm

@ArtificialZeng
Copy link
Contributor Author

Looks like you'll also need to sign the CLA, per the comment above.

Looks like you'll also need to sign the CLA, per the comment above.

I have already signed the CLA.

@bartekpacia
Copy link
Member

To me as a non-native English speaker, what's the difference?

@nate-thegrate nate-thegrate added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 20, 2024
@nate-thegrate
Copy link
Member

To me as a non-native English speaker, what's the difference?

Sorry @bartekpacia, didn't notice this comment till now!


The word "that" is used to create an essential clause, whereas "which" is used for making a non-essential clause.

The fact that Skia underpins Chrome and Android is good to know, but it could be removed from the sentence and you'd still be able to get the point across.

@auto-submit auto-submit bot merged commit 2f149da into flutter:master Sep 20, 2024
31 checks passed
@bartekpacia
Copy link
Member

Had no idea about this concept – TIL. Thanks! :)

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 21, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Sep 21, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Sep 21, 2024
flutter/flutter@7fe8237...2f149da

2024-09-20 875100501@qq.com Update README.md (flutter/flutter#155206)
2024-09-20 engine-flutter-autoroll@skia.org Roll Flutter Engine from 3fbe87c088d3 to 76b7abb5c853 (3 revisions) (flutter/flutter#155464)
2024-09-20 engine-flutter-autoroll@skia.org Roll Packages from d83d021 to f54fe93 (1 revision) (flutter/flutter#155459)
2024-09-20 engine-flutter-autoroll@skia.org Roll Flutter Engine from f5310c2345cc to 3fbe87c088d3 (1 revision) (flutter/flutter#155440)
2024-09-20 engine-flutter-autoroll@skia.org Roll Flutter Engine from 730f7124213f to f5310c2345cc (1 revision) (flutter/flutter#155436)
2024-09-20 engine-flutter-autoroll@skia.org Roll Flutter Engine from 90630957b33e to 730f7124213f (1 revision) (flutter/flutter#155431)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from d8ac05c3fb4f to 90630957b33e (1 revision) (flutter/flutter#155428)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from 0ed8af4d6e5d to d8ac05c3fb4f (2 revisions) (flutter/flutter#155425)
2024-09-19 engine-flutter-autoroll@skia.org Roll Packages from c7406b3 to d83d021 (3 revisions) (flutter/flutter#155421)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from 1d889dd5acac to 0ed8af4d6e5d (1 revision) (flutter/flutter#155419)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from af3e63a638f7 to 1d889dd5acac (1 revision) (flutter/flutter#155397)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from bc7d519a332c to af3e63a638f7 (1 revision) (flutter/flutter#155389)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from fe823a93f631 to bc7d519a332c (1 revision) (flutter/flutter#155387)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from 94f852d2348c to fe823a93f631 (5 revisions) (flutter/flutter#155385)
2024-09-19 engine-flutter-autoroll@skia.org Roll Flutter Engine from 15e9e3153266 to 94f852d2348c (2 revisions) (flutter/flutter#155376)
2024-09-19 49699333+dependabot[bot]@users.noreply.github.com Bump peter-evans/create-pull-request from 7.0.2 to 7.0.5 (flutter/flutter#155378)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC rmistry@google.com,stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
thejitenpatel pushed a commit to thejitenpatel/flutter that referenced this pull request Sep 26, 2024
This PR changes

> libraries like Skia (that underpins Chrome and Android)

to:

> libraries like Skia (which underpins Chrome and Android)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants