addKtlintCheckGitPreCommitHook always successfully even i broke the rule [pre-commit]
See original GitHub issuewhen using addKtlintCheckGitPreCommitHook
, my changed_files
with broken rule still got successfully then use addKtlintFormatGitPreCommitHook
facing still same issues
this is my build gradle on project level
buildscript {
repositories {
maven { url "https://plugins.gradle.org/m2/" }
}
dependecies {
classpath "org.jlleitschuh.gradle:ktlint-gradle:9.2.1"
}
}
allprojects {
apply plugin: "org.jlleitschuh.gradle.ktlint"
ktlint {
version = "0.35.0"
debug = true
verbose = true
android = true
outputToConsole = true
outputColorName = "RED"
enableExperimentalRules = false
}
}
when using ktlintCheck
my changed_files
ignored.
is that any problem with your PinternalKtlintGitFilter
?
Issue Analytics
- State:
- Created 3 years ago
- Comments:15 (1 by maintainers)
Top Results From Across the Web
black as pre-commit hook always fails my commits
We ask black to prevent any commits that include python files that haven't been formatted properly by black. It still makes sure that...
Read more >ktlint-gradle - Bountysource
BUILD SUCCESSFUL in 5s 7 actionable tasks: 7 executed ... folder- addKtlintCheckGitPreCommitHook always successfully even i broke the rule [pre-commit] $ 0.
Read more >Supported hooks - pre-commit
name-tests-test - verifies that test files are named correctly. no-commit-to-branch - don't commit to branch; requirements-txt-fixer - sorts entries in ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Should now be released: https://github.com/JLLeitschuh/ktlint-gradle/releases/tag/v10.3.0
Issue is actually already described in #562 and fixing PR is merged. However this is not released yet.