You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
⚠️ Is your feature request related to a problem? Please describe
ktfmtCheck binds to check as a dependency- causing build failures if things are not formatted correctly.
💡 Describe the solution you'd like
A configuration option to be exposed to disable this.
Context:
We at Google use ktfmt for a lot of our Kotlin projects, but not everyone wants their code to be formatted while working on a PR. Some folks are fine with it- others not so much. It would be significantly more convenient if we could have the formatting errors be warnings instead of failures during build time, and then rely on our commit hook to run it for failures.
From #46 , it seems this was a design choice. But the suggested workarounds are a bit too involved for a lot of user workflows. Those who work in the terminal don't have much issue with them- but if you're used to just running gradle tasks from the tasklist in Intellij/AS - then this is a bit of a smell. Sure, you can modify the run configuration- or create your own tasks and disable the ones registered automatically (which is what we currently do)- but it would make our lives much easier if we could just toggle it via a plugin or task configuration.
🤚 Do you want to develop this feature yourself?
Yes
No
We're comfortable creating a PR for this if you don't have the free cycles for it
The text was updated successfully, but these errors were encountered:
ktfmtCheck
binds tocheck
as a dependency- causing build failures if things are not formatted correctly.💡 Describe the solution you'd like
A configuration option to be exposed to disable this.
Context:
We at Google use
ktfmt
for a lot of our Kotlin projects, but not everyone wants their code to be formatted while working on a PR. Some folks are fine with it- others not so much. It would be significantly more convenient if we could have the formatting errors be warnings instead of failures during build time, and then rely on our commit hook to run it for failures.From #46 , it seems this was a design choice. But the suggested workarounds are a bit too involved for a lot of user workflows. Those who work in the terminal don't have much issue with them- but if you're used to just running gradle tasks from the tasklist in Intellij/AS - then this is a bit of a smell. Sure, you can modify the run configuration- or create your own tasks and disable the ones registered automatically (which is what we currently do)- but it would make our lives much easier if we could just toggle it via a plugin or task configuration.
🤚 Do you want to develop this feature yourself?
We're comfortable creating a PR for this if you don't have the free cycles for it
The text was updated successfully, but these errors were encountered: