New discussion on a proposed design doc for adding...
# development
c
New discussion on a proposed design doc for adding a
__defaults__
construct in BUILD files: https://github.com/pantsbuild/pants/discussions/15809
πŸŽ‰ 2
βœ… 2
πŸ”₯ 2
πŸ™Œ 2
🎣 for feedback πŸ˜‰
h
Ohhh I like this a lot, that you use a specific construct rather than a
python_sources
target overriding a
python_sources
target etc. There is still only one target owner per entity FYI this is similar to the
overrides
proposal https://docs.google.com/document/d/1bcQRJazx6ODKc0PZRUgPx_bqHc6XurpBawTMD2zhtAk/edit. I like this a lot more because the semantics are so clear: this solely impacts the default
c
Thanks. Yea saw the similarities with overrides as well.
w
love this, we’re using macros to propagate defaults to different areas of the codebass, this would help getting rid of them
πŸ‘ 2
b
Yeah super duper strong +10 from me 😭
❀️ 2
h
+100 from me. You avoid all the concerns I had with prior proposals for hierarchical metadata. The main thing I love is this only changes the default value; and you aren't creating any new targets
❀️ 1
w
this is really exciting! will try to take a look this afternoon.
πŸ‘ 1