-
Notifications
You must be signed in to change notification settings - Fork 27.4k
docs(faq): document the AngularJS/jqLite deprecation strategy #15819
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few minor improvements in the added text.
I noticed that we basically don't talk about deprecation in the rest of the FAQ. Maybe we should add this to the section about the versioning strategy, and add this new deprecation section as a subsection?
docs/content/misc/faq.ngdoc
Outdated
@@ -130,6 +130,15 @@ AngularJS 1.3 only supports jQuery 2.1 or above. jQuery 1.7 and newer might work | |||
but we don't guarantee that. | |||
|
|||
|
|||
### When will the deprecated features be removed from the library? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When are will the deprecated features be ...
docs/content/misc/faq.ngdoc
Outdated
@@ -130,6 +130,15 @@ AngularJS 1.3 only supports jQuery 2.1 or above. jQuery 1.7 and newer might work | |||
but we don't guarantee that. | |||
|
|||
|
|||
### When will the deprecated features be removed from the library? | |||
Most of the time we remove a deprecated feature in a next minor bump. For example, the `preAssignBindingsEnabled` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor version bump
docs/content/misc/faq.ngdoc
Outdated
@@ -130,6 +130,15 @@ AngularJS 1.3 only supports jQuery 2.1 or above. jQuery 1.7 and newer might work | |||
but we don't guarantee that. | |||
|
|||
|
|||
### When will the deprecated features be removed from the library? | |||
Most of the time we remove a deprecated feature in a next minor bump. For example, the `preAssignBindingsEnabled` | |||
`$compileProvider` method was defined in AngularJS `1.5.10`, deprecated in `1.6` and will be removed in `1.7`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll have to change this when 1.7.0 is released.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, maybe it'd be better to put sth else here, preferably something where we adhered to this strategy. :)
Makes sense. |
@Narretz PR updated, PTAL. I amended the commit as the diff wouldn't tell much anyway when the whole section is moved. |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs update.
What is the current behavior? (You can also link to an open issue here)
The deprecation strategy is not specified.
What is the new behavior (if this is a feature change)?
The deprecation strategy is specified.
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements
Other information:
Fixes #15282