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

Fixed grammar in compile.js (ngDoc) #10204

Closed
wants to merge 2 commits into from
Closed

Fixed grammar in compile.js (ngDoc) #10204

wants to merge 2 commits into from

Conversation

thinkingmedia
Copy link
Contributor

Changed "You may want to use this in production" to "You may want disable this in production". The phrase "to use" implies enabling where as it should be disabled in production.

Changed "You may want to use this in production" to "You may want disable this in production". The phrase "to use" implies enabling where as it should be *disabled* in production.
@googlebot
Copy link

CLAs look good, thanks!

@@ -870,7 +870,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
* * `ng-binding` CSS class
* * `$binding` data property containing an array of the binding expressions
*
* You may want to use this in production for a significant performance boost. See
* You may want disable this in production for a significant performance boost. See
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't it be "to disable" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yup, thanks. my bad.

changed "disable" to "to disable"
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.

None yet

3 participants