My recent attempt to address the missing link betw...
# developmentc
curved-television-6568
09/27/2022, 10:49 PMMy recent attempt to address the missing link between the python
requirements.txt👀 1
curved-television-6568
09/27/2022, 10:53 PMClosed PR: https://github.com/pantsbuild/pants/pull/16934
In order to address this, I started looking at how Pants could create synthesized targets (i.e. targets that isn’t declared in BUILD files): https://github.com/pantsbuild/pants/pull/16998
This in turn is a rather big feature (not in code, but in functionality offered) and as such, needs some care and consideration, hence this design doc: https://docs.google.com/document/d/1PPnXzUedm5mpPVuUdEm7WJppyiECHn28jIWOdp_uO3E/edit#
Input appreciated on that. 🙏
After this feature has landed, I can take another stab at implementing the dependency first intended by #16934 using synthetic targets instead.
curved-television-6568
09/27/2022, 10:54 PMcurved-television-6568
09/29/2022, 3:54 PMJust bumping this thread to keep it current.. 🙂
h
happy-kitchen-89482
09/29/2022, 4:12 PMAre we moving the discussion to https://docs.google.com/document/d/1PPnXzUedm5mpPVuUdEm7WJppyiECHn28jIWOdp_uO3E/edit# or to the issue?
c
curved-television-6568
09/29/2022, 5:36 PMwhere ever it makes most sense, I think. Perhaps depends on the nature of the comment? Fine either way by me.
curved-television-6568
09/29/2022, 5:37 PMjust posted the link to the issue here for completeness, I think 😛