Skip to content

Migrate to Markdown Javadoc for assertj-guava - #4298

Draft
scordio wants to merge 8 commits into
3.xfrom
guava-javadoc-markdown
Draft

scordio wants to merge 8 commits into
3.xfrom
guava-javadoc-markdown

Conversation

@scordio

@scordio scordio commented Jun 21, 2026

Copy link
Copy Markdown
Member

@scordio scordio added this to the 3.x milestone Jun 21, 2026
@scordio
scordio force-pushed the guava-javadoc-markdown branch 2 times, most recently from b464516 to 56a1296 Compare June 23, 2026 18:44
@scordio
scordio force-pushed the guava-javadoc-markdown branch 3 times, most recently from a27e038 to ce39f1f Compare July 1, 2026 21:45
@scordio
scordio marked this pull request as ready for review July 1, 2026 21:55
@scordio
scordio requested a review from Copilot July 1, 2026 21:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This pull request migrates assertj-guava’s Java API documentation from traditional Javadoc block comments to Markdown documentation comments, improving readability and maintainability (especially for examples and inline formatting) in line with the Java 25 toolchain.

Changes:

  • Converted Javadoc block comments (/** ... */) to Markdown documentation comments (/// ...) across assertj-guava API and error-message classes.
  • Reworked examples into fenced Markdown code blocks and replaced inline {@link ...} constructs with Markdown-style references.
  • Cleaned up some doc-adjacent code (e.g., removed now-unneeded imports where links were moved/rewritten).

Reviewed changes

Copilot reviewed 38 out of 38 changed files in this pull request and generated 10 comments.

Show a summary per file
File Description
assertj-guava/src/main/java/org/assertj/guava/util/ExceptionUtils.java Converted method/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/TableShouldHaveRowCount.java Converted factory method Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/TableShouldHaveColumnCount.java Converted factory method Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/TableShouldContainRows.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/TableShouldContainColumns.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/TableShouldContainCell.java Converted factory method Javadoc (incl. type params) to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/ShouldHaveSize.java Converted class and factory method Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/ShouldContainValues.java Converted class Javadoc to Markdown doc comments; adjusted TODO reference.
assertj-guava/src/main/java/org/assertj/guava/error/ShouldContainKeys.java Converted class and factory method Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldHaveUpperEndpointEqual.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldHaveLowerEndpointEqual.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldBeOpenedInTheUpperBound.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldBeOpenedInTheLowerBound.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldBeClosedInTheUpperBound.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeShouldBeClosedInTheLowerBound.java Converted constructor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldNotIntersect.java Converted class/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldNotEnclose.java Converted class/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldIntersectAnyOf.java Converted class/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldIntersect.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldEncloseAnyOf.java Converted class/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldEnclose.java Converted class/ctor Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/OptionalShouldBePresentWithValue.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/OptionalShouldBePresent.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/OptionalShouldBeAbsent.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/MultisetShouldContainTimes.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/MultisetShouldContainAtMostTimes.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/error/MultisetShouldContainAtLeastTimes.java Converted class Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/data/MapEntry.java Converted class and factory method Javadoc to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/api/TableAssert.java Converted method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/RangeSetAssert.java Converted extensive method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/RangeMapAssert.java Converted class/method Javadocs (incl. deprecation docs) to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/RangeAssert.java Converted method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/OptionalAssert.java Converted class/method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/MultisetAssert.java Converted method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/MultimapAssert.java Converted method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/InstanceOfAssertFactories.java Converted interface/field/method Javadocs to Markdown doc comments.
assertj-guava/src/main/java/org/assertj/guava/api/ByteSourceAssert.java Converted method Javadocs to Markdown doc comments and fenced examples.
assertj-guava/src/main/java/org/assertj/guava/api/Assertions.java Converted class and utility method Javadocs to Markdown doc comments and fenced examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread assertj-guava/src/main/java/org/assertj/guava/util/ExceptionUtils.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/RangeSetAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/RangeSetAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/TableAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/TableAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/TableAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/TableAssert.java Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 39 out of 39 changed files in this pull request and generated 7 comments.

Comment thread assertj-guava/src/main/java/org/assertj/guava/api/RangeMapAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/RangeSetAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/api/MultimapAssert.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/error/RangeSetShouldNotEnclose.java Outdated
Comment thread assertj-guava/src/main/java/org/assertj/guava/error/ShouldHaveSize.java Outdated
@scordio
scordio force-pushed the guava-javadoc-markdown branch from 9454980 to 7f1b8e4 Compare July 5, 2026 08:37
@scordio scordio mentioned this pull request Jul 5, 2026
@scordio
scordio force-pushed the guava-javadoc-markdown branch 2 times, most recently from 0a7cb33 to 495c562 Compare July 5, 2026 22:52
@scordio

scordio commented Jul 5, 2026

Copy link
Copy Markdown
Member Author

Switching to draft as there is something wrong with Spotless that I need to analyze.

@scordio

scordio commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Switching to draft as there is something wrong with Spotless that I need to analyze.

This is likely an Eclipse formatter bug, reported at:

scordio and others added 5 commits September 18, 2026 17:29
./mvnw -f assertj-guava/pom.xml -U org.openrewrite.maven:rewrite-maven-plugin:run --define rewrite.recipeArtifactCoordinates=org.openrewrite.recipe:rewrite-migrate-java:RELEASE --define rewrite.activeRecipes=org.openrewrite.java.migrate.lang.JavadocToMarkdownDocComment --define rewrite.exportDatatables=true
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@scordio
scordio force-pushed the guava-javadoc-markdown branch from 9d19d6b to 56c77ef Compare September 18, 2026 15:29
@testlens-app

testlens-app Bot commented Sep 18, 2026

Copy link
Copy Markdown

🚨 All tests passed but jobs failed 🚨

Failed Jobs without Test Failures

CI / Java 25 (macOS-latest)
CI / Java 25 (ubuntu-latest)
CI / Java 25 (windows-latest)
CI / Java 26
CI / Java 27
CI / Java 28

🏷️ Commit: 56c77ef
▶️ Tests: 40409 executed
⚪️ Checks: 19/19 completed


Learn more about TestLens at testlens.app/docs.

@github-actions

Copy link
Copy Markdown

Looks good. No mutations were possible for these changes.
See https://pitest.org/

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