quaint-telephone-89068
10/25/2022, 11:59 PMAllTargetsRequest
and ZipBinaryRequest
that solely existed to work around the lack of #12946. That is, it allowed us to "lazily" evaluate things, e.g. inside if statements, by doing await Get(ZipBinary, ZipBinaryRequest)
.
Now that we can do Get(ZipBinary, {})
, we should deprecate and then delete those old APIs. We only need the "singleton" rules that return ZipBinary
without needing to have a ZipBinaryRequest
. This change will make for less code and a tighter rule graph.
we should deprecateWe can do that by putting
warn_or_error()
inside the rules for ZipBinaryRequest
et al.
pantsbuild/pants