https://pantsbuild.org/ logo
Join Slack
Powered by
I just noticed a corner for `cli.alias` :thread:
# development
c
I just noticed a corner for 
cli.alias
🧵
👀 1
If you have a 
~/.pants.rc
file with a
Copy code
[cli.alias]
foo = bar
and in a projects 
pants.toml
Copy code
[cli.alias]
other = alias
then, the aliases from 
pants.toml
will not show up…
This is not only for cli.alias, obviously, but generically for nested config data not being merged.
h
specifically DictOption I think
c
yea
h
pretty sure Benjy fixed list options so that 
.add
and 
.remove
respect other config files
h
Yeah, we don't merge dicts across config files afaicr
lists do work
c
in some circumstances, this is probably the correct behaviour, though…
well, it’s a gotcha.
h
But that doesn't work with the nice TOML dicts we want to use. We'd need want to add first class support for 
[cli.alias.add]
, simliar to 
[cli].my_list_opt.add
➕ 1
💯 1
c
Great!
h
feel free to open an issue for that TOML syntax support. I don't think it would be very hard to do
c
Sure thing. Will get to it later today or tomorrow.
I started looking at it, but didn’t really grok where the 
+[…],-[…]
syntax was resolved into a single list object, so my mental map wasn’t clear as to if/how/where the same would work for 
+{…},-{…}
😉
h
I probably know this code best so I'll take a look at it
❤️ 2
3 Views
Open in Slack
PreviousNext
Powered by