fix: apply spotless formatting to gradle-plugin files#147
Conversation
Fixes spotless violations introduced by recent commits (ConfigParamGenerator, ProguardRulesGenerator, R8EliminationTest) — blank lines, inline comments, and parameter formatting per ktlint rules. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Review Summary by QodoApply spotless formatting to gradle-plugin files
WalkthroughsDescription• Remove unnecessary blank lines between when branches • Reformat inline comments to separate lines per ktlint • Adjust method parameter formatting for multi-line signatures • Fix trailing commas and indentation in method chains Diagramflowchart LR
A["Spotless Violations"] -->|Remove blank lines| B["ConfigParamGenerator.kt"]
A -->|Reformat comments| C["ProguardRulesGenerator.kt"]
A -->|Fix formatting| D["R8EliminationTest.kt"]
B --> E["Compliant Code"]
C --> E
D --> E
File Changes1. featured-gradle-plugin/src/main/kotlin/dev/androidbroadcast/featured/gradle/ConfigParamGenerator.kt
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewⓘ The new review experience is currently in Beta. Learn more |
CI Feedback 🧐A test triggered by this PR failed. Here is an AI-generated analysis of the failure:
|
There was a problem hiding this comment.
Pull request overview
Applies Spotless/ktlint formatting fixes to the Featured Gradle plugin code and tests to unblock CI checks (notably “Lint & API Check”) that are currently failing due to style violations.
Changes:
- Reformats
R8EliminationTestto satisfy ktlint (parameter wrapping, trailing commas, chaining layout). - Adjusts whitespace/comment placement in
ProguardRulesGeneratorto comply with ktlint rules. - Removes extra blank lines in
ConfigParamGenerator’swhenexpression formatting.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| featured-gradle-plugin/src/test/kotlin/dev/androidbroadcast/featured/gradle/R8EliminationTest.kt | Formatting-only refactor of the R8 elimination test helpers and invocations. |
| featured-gradle-plugin/src/main/kotlin/dev/androidbroadcast/featured/gradle/ProguardRulesGenerator.kt | Minor formatting/comment placement change in proguardLiteral’s when block. |
| featured-gradle-plugin/src/main/kotlin/dev/androidbroadcast/featured/gradle/ConfigParamGenerator.kt | Removes unnecessary blank lines within formatDefault() for ktlint compliance. |
Summary
ConfigParamGenerator.kt,ProguardRulesGenerator.kt, andR8EliminationTest.ktintroduced by recent commitsLint & API CheckTest plan
Lint & API Checkpasses🤖 Generated with Claude Code