Skip to content

Commit

Permalink
Add a warning to the bind() docs
Browse files Browse the repository at this point in the history
This will encourage new users to avoid it, and would have helped me. At
the start, I cargo-culted other projects that use bind() heavily without
knowing why.

Related: #1952

Closes #3608.

PiperOrigin-RevId: 166464352
  • Loading branch information
drigz authored and damienmg committed Aug 25, 2017
1 parent f6291af commit 054403d
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,10 @@ public Metadata getMetadata() {
}
/*<!-- #BLAZE_RULE (NAME = bind, TYPE = OTHER, FAMILY = Workspace)[GENERIC_RULE] -->
<em><p>Warning: use of <code>bind()</code> is not recommended. See "<a
href="https://github.com/bazelbuild/bazel/issues/1952">Consider removing bind</a>" for a long
discussion of its issues and alternatives.</p></em>
<p>Gives a target an alias in the <code>//external</code> package.</p>
<p>The <code>//external</code> package is not a "normal" package: there is no external/ directory,
Expand Down

0 comments on commit 054403d

Please sign in to comment.