hundreds-father-40403/04/2020, 2:18 PM
Migrating from pants.ini to pants.toml (Optional)
fancy-motherboard-2495603/04/2020, 2:27 PM
hundreds-father-40403/04/2020, 2:34 PM
so I’m trusting you did test it 🙂Yes, with both Pants and Toolchain. The most vague step is on fixing things manually, but it’s hard to prescribe the exact solutions because there are a couple permutations. My hope is that the email + the docs we just updated + editor validation and
error messages + the TOML spec + the 80% auto-generated changes from the script will be enough for people to see how to fix the remaining issues.
(As DMed), I keep going back and forth on trying to prescribe a solution for every possible issue but whenever I try to write this it gets very verbose and confusing. So, I think we rely on the above hints
aloof-angle-9161603/04/2020, 2:48 PM
file paths that may be hardcoded in a million other places is to make the config parsing code symlink-aware, and select which config file format to use depending on the symlink target's extension, instead of the symlink source. this would allow migrating
, then symlinking
, and having pants correctly use the TOML parser, even if provided
as a file path. since we would like to deprecate
, i think i'm comfortable with the mild dissonance. do you have any thoughts on that?
fancy-motherboard-2495603/04/2020, 2:51 PM
hundreds-father-40403/04/2020, 3:00 PM
rather than explicitly reading the config file. I know Twitter has a lot of places that refer to pants.ini, but I’m not sure if that’s a problem for other users too
fancy-queen-2073403/04/2020, 6:49 PM
aloof-angle-9161603/05/2020, 6:03 PM
instead of investing the time into “doing the right thing” and changing these links.if it's possible for us to change the links internally, then, we should probably be looking to decide on a
to agree on, and we should maybe bring this up with
so we can make sure other users don't get surprised.
Another approach that wouldn’t take symlinks to do would be to have the config loading code check for both the pants ini and toml files, looking for the provided one first, and warning with the external callsite if the requested one wasn’t the one that was found.
Adding that mechanism would do double duty because if we deprecate pants.ini, it’s precisely the best spot to add a warning about it.just all of this