aboutsummaryrefslogtreecommitdiffhomepage
path: root/site
diff options
context:
space:
mode:
authorGravatar Googler <noreply@google.com>2018-01-22 06:06:00 -0800
committerGravatar Copybara-Service <copybara-piper@google.com>2018-01-22 06:07:36 -0800
commita1a1e61387920c11abfecb15845401a527bab2fa (patch)
treeab61bd578afd64719183264df642c9a4f432d15e /site
parent0b29e64eb59c0ade10369583177591070a33f7f5 (diff)
Improve documentation for depsets.
PiperOrigin-RevId: 182767783
Diffstat (limited to 'site')
-rw-r--r--site/docs/skylark/depsets.md13
1 files changed, 5 insertions, 8 deletions
diff --git a/site/docs/skylark/depsets.md b/site/docs/skylark/depsets.md
index 7ebe2255bb..93ddbcd7ca 100644
--- a/site/docs/skylark/depsets.md
+++ b/site/docs/skylark/depsets.md
@@ -290,7 +290,8 @@ For safety, depsets with different orders cannot be merged with the `+` operator
unless one of them uses the default order; the resulting depset’s order is the
same as the left operand. Note that when two depsets of different order are
merged in this way, the child may appear to have had its elements rearranged
-when it is traversed via the parent.
+when it is traversed via the parent. **The `+` operator is deprecated, anyway;
+use the `transitive` argument instead.**
## Performance
@@ -365,10 +366,6 @@ and/or upcoming changes.
* Depset elements currently must have the same type, e.g. all ints or all
strings. This restriction will be lifted.
-* Depsets used to support `+` operator. It is now deprecated in favor of a new
- depset constructor (described in this document). This avoids confusion
- regarding how `+` treats direct elements vs children, and improves
- performance.
-
-* Both the `|` operator and the `.union()` methods are defined for depsets as
- a synonym for `+`. They will be going away.
+* A merge operation should be done by using the `transitive` argument in the
+ depset constructor. All other methods (`|` and `+` operators, and the
+ `union` method) are deprecated and will be going away.