Skip to content

Conversation

@kiftio
Copy link
Contributor

@kiftio kiftio commented Nov 12, 2025

What changes are you making?

Add automated code formatting with detekt, and add a pre-commit hook, to prevent issues like missing import optimization and inconsistent code formatting

Note: we may want to hold off to make the diff between main and development / prototype/development smaller

Changes

  1. Added detekt-formatting plugin
  2. Configured formatting rules (lib/detekt.config.yml). Uses detekt's default formatting rules (ktlint-based)
  3. Added pre-commit hook (.git/hooks/pre-commit.d/010-detekt-format)
  4. Fixed license header check script to exclude entire build/ directory
  5. Applied initial formatting

How to test

./gradlew detekt --auto-correct
./scripts/check_license_headers.rb
./gradlew clean test

Before you merge

Important


Checklist for releasing a new version
  • I have bumped the version number in the build.gradle file
  • I have updated the versions in the README.md for both Gradle and Maven.

Tip

See the Contributing documentation for instructions on how to publish a new version of the library.

@kiftio kiftio requested a review from a team as a code owner November 12, 2025 11:31
@kiftio kiftio force-pushed the prototype/development branch from c6be74a to 4d1aef8 Compare November 13, 2025 11:06
Base automatically changed from prototype/development to prototype/preview-1 November 14, 2025 17:40
@kiftio kiftio force-pushed the prototype/preview-1 branch from 3bf8c52 to 6fab619 Compare November 14, 2025 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant