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

docs($compile): remove meaningless sentence #15119

Closed
wants to merge 1 commit into from

Conversation

thorn0
Copy link
Contributor

@thorn0 thorn0 commented Sep 9, 2016

Closes #15109

@@ -269,12 +269,11 @@
* #### `scope`
* The scope property can be `true`, an object or a falsy value:
*
* * **falsy:** No scope will be created for the directive. The directive will use its parent's scope.
* * **falsy (default):** No scope will be created for the directive. The directive will use its parent's scope.
Copy link
Member

Choose a reason for hiding this comment

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

If you are adding (default) then change it to false (because falsy is not a concrete value 😃).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@gkalpak gkalpak closed this in d14c7f3 Sep 10, 2016
gkalpak pushed a commit that referenced this pull request Sep 10, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
petebacondarwin pushed a commit that referenced this pull request Nov 23, 2016
petebacondarwin pushed a commit that referenced this pull request Nov 24, 2016
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.

3 participants