Skip to content

feature/ktlint setup#39

Merged
nroi merged 4 commits into
mainfrom
feature/ktlint-setup
Jun 22, 2026
Merged

feature/ktlint setup#39
nroi merged 4 commits into
mainfrom
feature/ktlint-setup

Conversation

@nroi

@nroi nroi commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This PR is best reviewed commit-by-commit, because you most likely don't want to read through every change introduced by the formatter.

  • chore: add ktlint setup matching meshfed-release
  • style: reformat Kotlin sources with ktlintFormat

@nroi nroi force-pushed the feature/ktlint-setup branch from 87c4edd to cd32675 Compare June 17, 2026 14:35
@nroi nroi marked this pull request as ready for review June 17, 2026 15:14
@nroi nroi requested a review from grubmeshi June 17, 2026 15:14

@grubmeshi grubmeshi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

nroi and others added 4 commits June 22, 2026 09:15
Apply the org.jlleitschuh.gradle.ktlint plugin (v14.0.1, ktlint CLI
1.8.0) to all JVM modules via the root subprojects block, and align
.editorconfig with meshfed-release (trailing-comma enforcement, same
disabled-rule set). This adds build-time ktlintCheck/ktlintFormat tasks;
existing code is not yet reformatted.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Apply ktlintFormat across all modules so existing code complies with the
newly-added ktlint setup. ktlintCheck now passes; build is green.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
Add a PostToolUse hook (matching meshfed-release) that runs
`ktlint --format` on .kt/.kts files after Claude edits them, keeping
generated code compliant with the ktlint setup.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
@nroi nroi force-pushed the feature/ktlint-setup branch from cd32675 to 647cee8 Compare June 22, 2026 07:15
@nroi nroi merged commit a89a424 into main Jun 22, 2026
15 checks passed
@nroi nroi deleted the feature/ktlint-setup branch June 22, 2026 07:24
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.

2 participants