sds / overcommit / 42762484609 / 9
100%
master: 91%

LAST BUILD BRANCH: main
DEFAULT BRANCH: master
Ran
Files 446
Run time 7s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 71.835%. Remained the same
42762484609.9

push

github

web-flow 
Document the case-sensitive KtLint hook name (#927)

The configuration in #787 uses `Ktlint`, while the built-in hook is
named `KtLint`. With `required_executable: ktlint`, the misspelled name
becomes an ad hoc hook. It receives Git's original hook arguments, so a
pre-commit invocation runs Ktlint with no filenames and scans unmodified
files too.

Document the exact built-in name, give a minimal working configuration,
and explain why the alternative spelling behaves differently. Strengthen
the existing KtLint success example to require passing the applicable
filenames to the executable.

Fixes #787 through a configuration correction; no runtime change is
necessary.

Local reproduction with Ktlint 1.8.0 downloaded into `/tmp` and a
disposable Git repository:

- Commit an invalid `Bad.kt`, then stage only a valid `Good.kt` and
Overcommit configuration.
- With the report's `Ktlint` spelling, a real commit fails and reports
the unchanged `Bad.kt`.
- Change only the hook name to `KtLint`: the real commit succeeds and
does not report `Bad.kt`.

Validation: existing KtLint specs pass with the stronger argument
assertion; RuboCop passes. Full CI is required before merging.

Final verification: [run
42762226697](https://github.com/sds/overcommit/actions/runs/42762226697)
passed all 38 checks, including Linux/Windows Ruby 3.3, 3.4, and 4.0,
lint, CI, and all coverage statuses. All 20 completed jobs report zero
annotations.

5198 of 7236 relevant lines covered (71.84%)

68.08 hits per line

Source Files on job ruby3.4-windows-shard1of5 - 42762484609.9