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

Commit 65f800e

Browse files
committed
docs(angular.merge): add notes about support and lodash compatibility
Closes #16187 Closes #14512
1 parent 63796d3 commit 65f800e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/Angular.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -396,8 +396,8 @@ function extend(dst) {
396396
* sinceVersion="1.6.5"
397397
* This function is deprecated, but will not be removed in the 1.x lifecycle.
398398
* There are edge cases (see {@link angular.merge#known-issues known issues}) that are not
399-
* supported by this function. We suggest
400-
* using [lodash's merge()](https://lodash.com/docs/4.17.4#merge) instead.
399+
* supported by this function. We suggest using another, similar library for all-purpose merging,
400+
* such as [lodash's merge()](https://lodash.com/docs/4.17.4#merge).
401401
*
402402
* @knownIssue
403403
* This is a list of (known) object types that are not handled correctly by this function:
@@ -406,6 +406,8 @@ function extend(dst) {
406406
* - [`CanvasGradient`](https://developer.mozilla.org/docs/Web/API/CanvasGradient)
407407
* - AngularJS {@link $rootScope.Scope scopes};
408408
*
409+
* `angular.merge` also does not support merging objects with circular references.
410+
*
409411
* @param {Object} dst Destination object.
410412
* @param {...Object} src Source object(s).
411413
* @returns {Object} Reference to `dst`.

0 commit comments

Comments
 (0)