high-yak-85899
05/11/2023, 5:25 PM./pants fmt
log a warning when changes are made? Isn't that point of running formatters?happy-kitchen-89482
05/11/2023, 5:28 PMhigh-yak-85899
05/11/2023, 5:30 PMbusy-vase-39202
05/11/2023, 5:37 PMWARN
definitely implies "a problem you need to be aware of" to me. For expected behavior, INFO
does seem more appropriate. This has a bunch of interesting takes, and seem all to lean toward INFO
as most befitting this kind of situation.happy-kitchen-89482
05/11/2023, 5:41 PMhigh-yak-85899
05/11/2023, 5:51 PMfmt
says it made changes while git status
shows nothing?
11:45:13.64 [WARN] Completed: Format with isort - isort made changes.
<a bunch of files here>
+ isort made changes.
(3.8.10) ~/devel/monorepo (isort)$ gs
On branch isort
Your branch is ahead of 'master' by 2 commits.
(use "git push" to publish your local commits)
nothing to commit, working tree clean
bitter-ability-32190
05/11/2023, 5:52 PMhigh-yak-85899
05/11/2023, 5:59 PM--only=isort
for thisbroad-processor-92400
05/11/2023, 8:44 PMhigh-yak-85899
05/12/2023, 11:22 PMa PR to that effect would be very welcome!This one should be easy enough for me to tackle. I'll try to make it happen this weekend.
busy-vase-39202
05/12/2023, 11:27 PM