Quick feedback, it would be even more helpful if t...
# developmentc
curved-television-6568
10/13/2021, 8:11 AMQuick feedback, it would be even more helpful if this deprecation notice included information about which BUILD file the deprecated target was found in (or the target’s address even)
Copy code
10:09:39.57 [WARN] DEPRECATED: the target name python_library will be removed in version 2.9.0.dev0.
Use `python_sources` instead, which behaves the same.
To automate fixing this, download <https://github.com/pantsbuild/pants/blob/release_2.8.0.dev4/build-support/migration-support/rename_targets_pants28.py>, then run `python3 rename_targets_pants28.py --help` for instructions.h
hundreds-father-404
10/13/2021, 1:27 PMThe reason I didn't do that it is would result in hundreds of warnings. One per target. That seemed like a bad idea. Wdyt?
c
curved-television-6568
10/13/2021, 1:38 PMOh, right, yeah… hmmm… that is undesireable too, of course. Isn’t there a limit for a certain class of warnings, that they get silenced if there are loads of them?
curved-television-6568
10/13/2021, 1:40 PMOr if this is simply custom code, with a warning logg, could there be a counter, so we only log the first X (small number) instances, then after that drop the specific information and keep this generic template, so it doesn’t spam…
curved-television-6568
10/13/2021, 1:40 PMI do get this warning on the pants repo, but I haven’t gone fishing for where it comes from.. 😉
h
hundreds-father-404
10/13/2021, 2:05 PMWe don't currently have a way to do that. We switched from using
warningslogging.warninghundreds-father-404
10/13/2021, 2:05 PMI think where I want to spend more energy instead is
./pants mend./pants mend👍 1
c
curved-television-6568
10/13/2021, 2:09 PMYeah, it’s not worth spending a lot of effort on.