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