Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Update controller.ngdoc fixes grammar #11000

Closed
wants to merge 1 commit into from

Conversation

rwuebker
Copy link
Contributor

@rwuebker rwuebker commented Feb 6, 2015

Improper use of the word 'which'.

BEFORE: The following example uses a Controller to add a method to the scope, which doubles a number:
AFTER: The following example uses a Controller to add a method, which doubles a number, to the scope:

@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@rwuebker
Copy link
Contributor Author

rwuebker commented Feb 6, 2015

Hello,

I have signed the agreement.

Thanks,
Rick

On Fri, Feb 6, 2015 at 3:05 PM, googlebot [email protected] wrote:

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source
project, in which case you'll need to sign a Contributor License Agreement
(CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub
username or you're using a different email address. Check the information
on your CLA https://cla.developers.google.com/clas or see this help
article on setting the email on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Once you've done that, please reply here to let us know. If you signed the
CLA as a corporation, please let us know the company's name.


Reply to this email directly or view it on GitHub
#11000 (comment).

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Feb 6, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants