https://pantsbuild.org/ logo
#development
Title
# development
w

witty-crayon-22786

01/27/2022, 9:20 PM
💯 2
🔥 3
ideally, use a pants-devel@ member email to comment directly, but otherwise: reply on the ticket (or here)
c

curved-television-6568

01/27/2022, 9:33 PM
Love it. Halfway through I thought that a field marked as parametrizable could optionally have a method for producing the name for the address, in case the data is unusable as-is. But then were the configuration construct presented, and I think that is a workable approach to rid that issue wholesale. Great proposal!
❤️ 1
w

witty-crayon-22786

01/27/2022, 10:13 PM
Halfway through I thought that a field marked as parametrizable could optionally have a method for producing the name for the address, in case the data is unusable as-is. But then were the configuration construct presented, and I think that is a workable approach to rid that issue wholesale.
ah, yea: that was the conclusion that i came to… i started to go down a rabbit hole around serialization of parameterizable fields in addresses, but in the end you want the address to be pretty short anyway
1
based on feedback from @curved-television-6568 and @hundreds-father-404, and due to wanting to support complex fields (as highlighted by @happy-kitchen-89482), i’ve adjusted the design to use explicit
parameterize("a", "b")
syntax. probably ready for another look. thanks!
👀 1
c

curved-television-6568

01/29/2022, 4:55 PM
Even better. Love this direction.
❤️ 1
💯 1
w

witty-crayon-22786

01/31/2022, 7:41 PM
i think that the design is almost finished, with the exception of two open questions: 1. the address syntax, and how it interacts with generators 2. ensuring that configuration and #13767 will play nice thanks for the help so far! from an implementation perspective, i believe that all of Parameterization can land without implementing Configuration, so i’ll be starting that today. but before landing Parameterization (and before cutting
2.10.0rc0
), we need to nail down the syntax: more discussion welcome. current thread on the doc, and in slack