docs(Scope): clarify naming in the $watch example

[pkozlowski-opensource]

Fixes #8254

[mary-poppins]

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

• Uses the issue template (#8271)

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!

[Narretz]

Internally, this function is sometimes called the watchAction, but change listener makes sense in this example.

[petebacondarwin]

LGTM although I would check the line lengths :-)

[pkozlowski-opensource]

@petebacondarwin I guess we should have a rule in jshint / jscs to check for the line length if we want to enforce those, otherwise there will be always someone not prudent enough :-)

[caitp]

I believe we are enforcing a 200 col limit, globally.

[petebacondarwin]

Oh I thought it was 100
On 24 Jul 2014 18:15, "Caitlin Potter" [email protected] wrote:

I believe we are enforcing a 200 col limit, globally.

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

[caitp]

I think you should try to keep code under 100 cols, but jshint-base in the root of the tree has maxlen: 200 and is inherited by all of the different sub-configs in the tree.