Skip to content

Add compatibility dashboards to the documentation site#461

Merged
bertysentry merged 11 commits intomainfrom
426-add-proper-reports-in-documentation-about-the-compatibility-tests
Apr 29, 2026
Merged

Add compatibility dashboards to the documentation site#461
bertysentry merged 11 commits intomainfrom
426-add-proper-reports-in-documentation-about-the-compatibility-tests

Conversation

@bertysentry
Copy link
Copy Markdown
Contributor

Summary

  • convert the homepage and compatibility page to Velocity templates for live compatibility reporting
  • generate a compatibility summary XML file from Failsafe reports during the site build
  • add documentation-site compatibility progress bars and per-test heatmaps

Validation

  • mvn site
  • mvn verify site

@bertysentry bertysentry linked an issue Apr 28, 2026 that may be closed by this pull request
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2ec8510ed2

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/site/java/io/jawk/site/CompatibilitySummaryGenerator.java Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds “live” compatibility dashboards to the Maven documentation site by generating an aggregate compatibility summary from Failsafe reports and rendering it via Velocity-backed Markdown templates.

Changes:

  • Convert index.md and compatibility.md into Velocity templates (.md.vm) that render progress bars and per-test heatmaps from a generated XML summary.
  • Add CSS styles for compatibility progress bars, legends, and heatmaps.
  • Introduce a Java-based generator to produce compatibility-summary.xml during the site build, wired via exec-maven-plugin.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/site/site.xml Renames the navigation item to “Compatibility and Compliance”.
src/site/resources/css/site.css Adds styles for compatibility progress bars and heatmaps.
src/site/markdown/index.md.vm New Velocity template homepage rendering compatibility summary bars.
src/site/markdown/index.md Removes static homepage in favor of the Velocity template.
src/site/markdown/compatibility.md.vm New Velocity template compatibility dashboard with per-test heatmaps.
src/site/markdown/compatibility.md Removes static compatibility page in favor of the Velocity template.
src/site/markdown/cli-reference.md Adds documentation about tuple serialization compatibility.
src/site/java/io/jawk/site/CompatibilitySummaryGenerator.java Adds a generator that aggregates Failsafe XML into a site-consumable summary XML.
pom.xml Adds an exec-maven-plugin execution intended to generate the summary XML before the site runs.

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

Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread src/site/java/io/jawk/site/CompatibilitySummaryGenerator.java Outdated
Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread src/site/resources/css/site.css
Comment thread pom.xml
Comment thread src/site/java/io/jawk/site/CompatibilitySummaryGenerator.java Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 8 comments.


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

Comment thread src/site/resources/css/site.css
Comment thread src/site/resources/css/site.css Outdated
Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java Outdated
Comment thread pom.xml Outdated
Comment thread src/site/markdown/index.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 4 comments.


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

Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 4 comments.


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

Comment thread pom.xml Outdated
Comment thread src/site/markdown/index.md.vm Outdated
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/resources/css/site.css
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 6 comments.


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

Comment thread src/site/markdown/index.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread pom.xml Outdated
Comment thread pom.xml Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 3 comments.


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

Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread src/site/markdown/index.md.vm Outdated
Comment thread src/site/markdown/compatibility.md.vm Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 9 out of 9 changed files in this pull request and generated 6 comments.


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

Comment thread src/site/resources/css/site.css
Comment thread src/it/java/io/jawk/site/CompatibilitySummaryGenerator.java
Comment thread pom.xml
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
Comment thread src/site/markdown/compatibility.md.vm
@bertysentry bertysentry merged commit d147a06 into main Apr 29, 2026
9 checks passed
@bertysentry bertysentry deleted the 426-add-proper-reports-in-documentation-about-the-compatibility-tests branch April 29, 2026 16:41
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.

Add proper reports in documentation about the compatibility tests

2 participants