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

docs: fix a grammatical error in lazy-loading-ngmodules guide #42008

Closed
wants to merge 1 commit into from

Conversation

Sliqric7053
Copy link
Contributor

@Sliqric7053 Sliqric7053 commented May 9, 2021

correct a misspelled word

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

There is a grammatical error in the lazy-loading-ngmodules guide page.

Issue Number: N/A

What is the new behavior?

The grammatical error in the lazy-loading-ngmodules guide page has been corrected.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@google-cla
Copy link

google-cla bot commented May 9, 2021

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

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no label May 9, 2021
@pullapprove pullapprove bot requested a review from atscott May 9, 2021 06:03
@Sliqric7053
Copy link
Contributor Author

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

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.

What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

@googlebot I signed it!

@google-cla google-cla bot added cla: yes and removed cla: no labels May 9, 2021
Copy link
Contributor

@MrJithil MrJithil left a comment

Choose a reason for hiding this comment

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

Change the PR header to explanatory.

E.g.: docs: fix a grammatical error in Lazy-loading feature.

Also, it's good to have some content in the current behavior and new behavior sections. It will be helpful for contributors to understand the relevance of this change at a glance.

@Sliqric7053
Copy link
Contributor Author

Sliqric7053 commented May 9, 2021 via email

@Sliqric7053 Sliqric7053 changed the title docs: fix typo docs: fix a grammatical error in lazy-loading-ngmodules guide May 9, 2021
@Sliqric7053
Copy link
Contributor Author

Change the PR header to explanatory.

E.g.: docs: fix a grammatical error in Lazy-loading feature.

Also, it's good to have some content in the current behavior and new behavior sections. It will be helpful for contributors to understand the relevance of this change at a glance.

Thanks for your feedback, I've updated the PR : )

@petebacondarwin
Copy link
Member

Change the PR header to explanatory.
E.g.: docs: fix a grammatical error in Lazy-loading feature.
Also, it's good to have some content in the current behavior and new behavior sections. It will be helpful for contributors to understand the relevance of this change at a glance.

Thanks for your feedback, I've updated the PR : )

Sadly this is not exactly accurate. It doesn't really matter what the PR title and description are (other than to make it easier for us to triage and review). The more important point is that the commit message itself has to follow our commit requirements. Please take a look and let me know if you need help amending the commit.

@petebacondarwin petebacondarwin added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews comp: docs labels May 9, 2021
@ngbot ngbot bot modified the milestone: Backlog May 9, 2021
@Sliqric7053
Copy link
Contributor Author

Change the PR header to explanatory.
E.g.: docs: fix a grammatical error in Lazy-loading feature.
Also, it's good to have some content in the current behavior and new behavior sections. It will be helpful for contributors to understand the relevance of this change at a glance.

Thanks for your feedback, I've updated the PR : )

Sadly this is not exactly accurate. It doesn't really matter what the PR title and description are (other than to make it easier for us to triage and review). The more important point is that the commit message itself has to follow our commit requirements. Please take a look and let me know if you need help amending the commit.

Thanks for the feedback, yes please - should I discard the PR and create a new one with a proper commit message, or can I amend the current one? If the latter, I would appreciate your assistance in this regard.

Thanks again.

@petebacondarwin
Copy link
Member

You should not need to start a new PR, although I suspect you made this change via the GitHub UI, rather than cloning the repository locally. One approach would be:

git clone https://github.com/Sliqric7053/angular
git checkout patch-1
git commit --amend -m"docs: fix a grammatical error in lazy-loading-ngmodules guide"
git push --force-with-lease

@petebacondarwin
Copy link
Member

If you struggle to do this, I could probably fix it up for you from here.

@google-cla
Copy link

google-cla bot commented May 10, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no and removed cla: yes labels May 10, 2021
@petebacondarwin
Copy link
Member

Thanks for the update @Sliqric7053 - unfortunately your local git was setup with the following email address: riccirich@Riccis-MacBook-Pro.local, which is not what you used to sign the CLA.

I think you could fix this by running:

git config --global user.email "email@example.com"

with your actual email address, and then amending the commit once again:

git commit --amend --reuse-message=HEAD
git push --force-with-lease

@google-cla
Copy link

google-cla bot commented May 10, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@petebacondarwin
Copy link
Member

And finally you just need to post a comment with:

@googlebot I fixed it.

@Sliqric7053
Copy link
Contributor Author

Thanks for the update @Sliqric7053 - unfortunately your local git was setup with the following email address: riccirich@Riccis-MacBook-Pro.local, which is not what you used to sign the CLA.

I think you could fix this by running:

git config --global user.email "email@example.com"

with your actual email address, and then amending the commit once again:

git commit --amend --reuse-message=HEAD
git push --force-with-lease

Hmm.. I've updated the git username/email as advised, but I think git is still picking up my default (incorrect details).

image

@angular angular deleted a comment from google-cla bot May 10, 2021
@google-cla

This comment has been minimized.

@petebacondarwin
Copy link
Member

Try this: #42008 (comment)

@Sliqric7053
Copy link
Contributor Author

@googlebot I fixed it.

@googlebot I fixed it.

@google-cla
Copy link

google-cla bot commented May 10, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@Sliqric7053
Copy link
Contributor Author

Try this: #42008 (comment)

image

The bot says.. No : (

@petebacondarwin
Copy link
Member

😭

I can see that the commit doesn't seem to have changed: https://github.com/angular/angular/commit/8cbf84a948d06fbcb45bb87c89c014a75151d0c5.patch

Perhaps try:

git commit --amend --author="Richard Sithole <your@email.address>"

@Sliqric7053
Copy link
Contributor Author

😭

I can see that the commit doesn't seem to have changed: https://github.com/angular/angular/commit/8cbf84a948d06fbcb45bb87c89c014a75151d0c5.patch

Perhaps try:

git commit --amend --author="Richard Sithole <your@email.address>"

Finally, bot seems to be happy :)

Just awaiting these tasks..

image

Thanks so much for your help and patience 💯 : )

Copy link
Member

@petebacondarwin petebacondarwin left a comment

Choose a reason for hiding this comment

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

Reviewed-for: global-docs-approvers

@petebacondarwin petebacondarwin removed the request for review from atscott May 10, 2021 10:55
@petebacondarwin petebacondarwin added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels May 10, 2021
@alxhub alxhub closed this in e2f09fe May 10, 2021
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jun 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants