Conversation
b464516 to
56a1296
Compare
a27e038 to
ce39f1f
Compare
There was a problem hiding this comment.
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 (/// ...) acrossassertj-guavaAPI 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.
9454980 to
7f1b8e4
Compare
0a7cb33 to
495c562
Compare
|
Switching to draft as there is something wrong with Spotless that I need to analyze. |
This is likely an Eclipse formatter bug, reported at: |
495c562 to
51bea58
Compare
9f0478e to
9d19d6b
Compare
./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>
9d19d6b to
56c77ef
Compare
🚨 All tests passed but jobs failed 🚨Failed Jobs without Test Failures❌ CI / Java 25 (macOS-latest) 🏷️ Commit: 56c77ef Learn more about TestLens at testlens.app/docs. |
|
Looks good. No mutations were possible for these changes. |
This migrates the
assertj-guavaJavadoc to the Markdown syntax.Relates to:
Blocked by:
///Markdown line in code block merged with next line into/// ///eclipse-jdt/eclipse.jdt.core#5220