#154: ktlint implementation

ARK-Builders / ARK-Navigator

Android app for navigation through your data

MIT License

15 stars 15 forks source link

#154: ktlint implementation #181

Closed kirillt closed 2 years ago

kirillt commented 2 years ago

@CloudLevi

Several Gradle versions seem to be mentioned (7.0.2 and 7.0.4)
File .editorconfig should be used to specify rules of formatting
Is pre-commit hook really working? (I've tried it on Linux and it looks possible to commit non-nformatted code without problem)

CloudLevi commented 2 years ago

@kirillt about gradle versions: In build.gradle, we define the version for android gradle plugin. In gradle-wrapper.properties, we define the gradle distribution version, which are different things. The gradle-wrapper.properties file was changed automatically upon updating the gralde version, so it should be the recommended supported verion of distribution for our gradle version, which is 7.0.4

kirillt commented 2 years ago

Optimized pre-commit hooks (ktlint is run only if there are changed .kt files). Optimized GitHub Actions workflows.