RFR: 1690: Make jcheck cli able to use conf from workspace or other commit
Erik Joelsson
erikj at openjdk.org
Wed Nov 30 00:02:27 UTC 2022
On Tue, 29 Nov 2022 22:43:38 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> It case 5 really worth the complexity? The `--working-tree` option is equivalent to `--conf-file .jcheck/conf`. As for `--staged`, I can imagine that it might be an occasionally useful "nice to have", but how often will developers really want to run `jcheck` against the version of `.jcheck/conf` in the index?
The idea with 5 is that the user won't have to commit before performing a whitespace check. I find it rather cumbersome to have to commit before I can verify jcheck.
I agree that `--conf-working-tree` is equivalent to `--conf-file .jcheck/conf` so we don't really need it. OTOH, I figured it would be pretty simple to implement.
-------------
PR: https://git.openjdk.org/skara/pull/1428
More information about the skara-dev
mailing list