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

Fix wording in 'Understanding the Controller Component' doc#3454

Closed
Nitrodist wants to merge 1 commit into
angular:masterfrom
Nitrodist:fix-wording-in-controller-dev-guide
Closed

Fix wording in 'Understanding the Controller Component' doc#3454
Nitrodist wants to merge 1 commit into
angular:masterfrom
Nitrodist:fix-wording-in-controller-dev-guide

Conversation

@Nitrodist
Copy link
Copy Markdown
Contributor

Fixes the wording so that it goes from:

'Do not use controllers for to run stateless or stateful code shared across controllers'

to:

'Do not use controllers for stateless or stateful code sharing across controllers'

@mary-poppins
Copy link
Copy Markdown

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

Wording has been changed in two of the examples to read naturally:

From: 'Do not use controllers for to run stateless or stateful code
shared across controllers'

To: 'Do not use controllers for stateless or stateful code sharing
across controllers'
@Nitrodist
Copy link
Copy Markdown
Contributor Author

Signed the CLA as Mark Campbell and updated the commit message.

petebacondarwin pushed a commit that referenced this pull request Aug 3, 2013
Wording has been changed in two of the examples to read naturally.
For example:

From: 'Do not use controllers for to run stateless or stateful code
shared across controllers'

To: 'Do not use controllers for sharing stateless or stateful code
across controllers'

Closes #3454
@petebacondarwin
Copy link
Copy Markdown
Contributor

Thanks Mark

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants