Skip to content

build: add Checkstyle reporter for unused imports and trailing whitespace - #1203

Merged
kabir merged 1 commit into
a2aproject:mainfrom
Kudwa-Abhishek:issue-139
Oct 6, 2026
Merged

kabir merged 1 commit into
a2aproject:mainfrom
Kudwa-Abhishek:issue-139

Conversation

@Kudwa-Abhishek

Copy link
Copy Markdown
Contributor

Summary

  • Added Checkstyle to the Maven build as a report-only validation step.

  • Added checks for:

    • Unused imports
    • Trailing whitespace
  • Enabled the checks for both main and test Java sources.

  • Removed the unused imports identified by Checkstyle.

  • Kept violations non-blocking so existing formatting issues do not introduce unrelated changes.

Testing

Ran:

mvn -B org.apache.maven.plugins:maven-checkstyle-plugin:3.5.0:check \
  -Dcheckstyle.config.location=src/checkstyle/checkstyle.xml \
  -Dcheckstyle.includeTestSourceDirectory=true \
  -Dcheckstyle.failOnViolation=false

Result:

BUILD SUCCESS

Also verified:

git diff --check

Result: no output / no whitespace errors introduced by this change.

Related issue: #139

@kabir

kabir commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

Thank you @Kudwa-Abhishek !

@kabir
kabir merged commit fc2e96e into a2aproject:main Oct 6, 2026
14 checks passed
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