Skip to content

chore: allow JUnit lifecycle methods without Javadocs - #2190

Open
thekarthikh wants to merge 1 commit into
googleapis:mainfrom
thekarthikh:codex/google-http-test-javadocs
Open

thekarthikh wants to merge 1 commit into
googleapis:mainfrom
thekarthikh:codex/google-http-test-javadocs

Conversation

@thekarthikh

Copy link
Copy Markdown

Fixes #1267

The Checkstyle configuration currently requires Javadocs for public JUnit lifecycle methods such as @before and @after, producing warnings in shared test classes. This change allows those annotations in both Javadoc checks while preserving the existing checks for production methods.

Validation:

  • mvn -pl google-http-client-test -am -DskipTests checkstyle:check

The issue is open and unassigned. The repository contribution guide requests the Accepting Contributions label; this PR is submitted for maintainer review of that contribution gate.

@thekarthikh
thekarthikh requested a review from a team as a code owner September 30, 2026 22:39
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Sep 30, 2026
@google-cla

google-cla Bot commented Sep 30, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't require javadoc for tests

1 participant