bazel-skylib icon indicating copy to clipboard operation
bazel-skylib copied to clipboard

Update README.md

Open BoleynSu opened this issue 3 years ago • 4 comments
trafficstars

If we will remove new_sets, then we should deprecate it instead of sets.

BoleynSu avatar Jul 23 '22 07:07 BoleynSu

I think the docs are correct and sets are the ones deprecated.

comius avatar Sep 13 '22 09:09 comius

Then we should fix the comments in the source file?

BoleynSu avatar Sep 13 '22 10:09 BoleynSu

See https://github.com/bazelbuild/bazel-skylib/blob/main/lib/old_sets.bzl

BoleynSu avatar Sep 13 '22 10:09 BoleynSu

@tetromino it makes sense to point users to the sets.bzl, right? And get over the new_ and old_, fiasco.

comius avatar Sep 15 '22 17:09 comius