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

refactor(Null safety): fix typo about number of core design principles #5154

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

dancer1325
Copy link
Contributor

@dancer1325 dancer1325 commented Sep 6, 2023

  • Fix a typo about number of core design principles mentioned afterwards

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
  • This PR doesn’t contain automatically generated corrections or text (Grammarly, LLMs, and similar).
  • This PR follows the Google Developer Documentation Style Guidelines — for example, it doesn’t use i.e. or e.g., and it avoids I and we (first person).
  • This PR uses semantic line breaks of 80 characters or fewer.
Contribution guidelines:
  • See our contributor guide for general expectations for PRs.
  • Larger or significant changes should be discussed in an issue before creating a PR.
  • Code changes should generally follow the Dart style guide and use dart format.
  • Updates to code excerpts indicated by <?code-excerpt need to be updated in their source .dart file as well.

@google-cla
Copy link

google-cla bot commented Sep 6, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@atsansone atsansone added cla: no Contributor has not signed the Contributor License Agreement review.await-update Awaiting Updates after Edits labels Sep 6, 2023
Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

Before we can review and incorporate the change, you'll have to sign the Google CLA as outlined in the earlier PR comment with the same email as you committed with: #5154 (comment)

Once that is complete, we'd be happy to take another look. Thank you!

@dancer1325
Copy link
Contributor Author

/rerun

@dancer1325
Copy link
Contributor Author

Thanks for the PR!

Before we can review and incorporate the change, you'll have to sign the Google CLA as outlined in the earlier PR comment with the same email as you committed with: #5154 (comment)

Once that is complete, we'd be happy to take another look. Thank you!

Hi @parlough

  • Already signed the CLA -- Sorry for that ... --
  • I was not able to check that all was fine, since I couldn't relaunch the github action which triggers the 'build.yml'

@google-cla google-cla bot added cla: yes Contributor has signed the Contributor License Agreement and removed cla: no Contributor has not signed the Contributor License Agreement labels Sep 19, 2023
@parlough
Copy link
Member

parlough commented Sep 19, 2023

Thanks for the response and I'm really sorry about the back and forth... I appreciate the PR and your patience. I re-ran the CLA check and it seems it's still failing.

Check the workflow for more information (https://github.com/dart-lang/site-www/pull/5154/checks?check_run_id=16833416591), but it seems the commit was made with your hidden GitHub email. The workflow failure explains more, but you may need to add that to your CLA or recommit with the email you signed the CLA with.

@parlough parlough removed the review.await-update Awaiting Updates after Edits label Sep 20, 2023
Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks for this, looks good to me!

Sorry about the delay and the CLA issues.

@parlough parlough merged commit d95d11e into dart-lang:main Sep 20, 2023
9 checks passed
MaryaBelanger pushed a commit that referenced this pull request Nov 14, 2023
#5154)

* Fix a typo about number of core design principles mentioned afterwards

---

- [x] I’ve reviewed the contributor guide and applied the relevant
portions to this PR.
- [x] This PR doesn’t contain automatically generated corrections or
text (Grammarly, LLMs, and similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.

<details>
  <summary>Contribution guidelines:</summary><br>

- See our [contributor
guide](https://github.com/dart-lang/site-www/blob/main/CONTRIBUTING.md)
for general expectations for PRs.
- Larger or significant changes should be discussed in an issue before
creating a PR.
- Code changes should generally follow the [Dart style
guide](https://dart.dev/effective-dart) and use `dart format`.
- Updates to [code
excerpts](https://github.com/dart-lang/site-shared/blob/main/doc/code-excerpts.md)
indicated by `<?code-excerpt` need to be updated in their source `.dart`
file as well.
</details>

Co-authored-by: Anthony Sansone <[email protected]>
atsansone added a commit to atsansone/site-www that referenced this pull request Jan 26, 2024
dart-lang#5154)

* Fix a typo about number of core design principles mentioned afterwards

---

- [x] I’ve reviewed the contributor guide and applied the relevant
portions to this PR.
- [x] This PR doesn’t contain automatically generated corrections or
text (Grammarly, LLMs, and similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.

<details>
  <summary>Contribution guidelines:</summary><br>

- See our [contributor
guide](https://github.com/dart-lang/site-www/blob/main/CONTRIBUTING.md)
for general expectations for PRs.
- Larger or significant changes should be discussed in an issue before
creating a PR.
- Code changes should generally follow the [Dart style
guide](https://dart.dev/effective-dart) and use `dart format`.
- Updates to [code
excerpts](https://github.com/dart-lang/site-shared/blob/main/doc/code-excerpts.md)
indicated by `<?code-excerpt` need to be updated in their source `.dart`
file as well.
</details>

Co-authored-by: Anthony Sansone <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants