Skip to content

Doc Combination & Upgrade Guide#1211

Merged
jdaugherty merged 7 commits intoapache:7.0.xfrom
jdaugherty:doc-combination
Mar 19, 2026
Merged

Doc Combination & Upgrade Guide#1211
jdaugherty merged 7 commits intoapache:7.0.xfrom
jdaugherty:doc-combination

Conversation

@jdaugherty
Copy link
Contributor

I've taken the upgrade guide from: apache/grails-core#15476 and created a combined docs. I've also improved the styling of the root guide page.

Copy link
Contributor

@matrei matrei left a comment

Choose a reason for hiding this comment

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

There seems to be some duplicate heading ids now that the documentation is aggregated.
Should be an easy fix by prefixing the ids with the plugin like core-introduction.

WARNING: ../../../plugin-core/docs/src/docs/introduction.adoc: line 21: id assigned to section already in use: introduction

The index.adoc header links were hardcoded to Spring Security 4.0.x and
Spring Framework 4.1.x. Replace with AsciiDoc attributes populated from
the BOM-resolved version in docs/build.gradle. Also add a NOTE comment
about configuration-time dependency resolution in the docs build.

Assisted-by: Claude Code <Claude@Claude.ai>
Each plugin docs/build.gradle applied reproducible-config.gradle twice:
once at the top level and again inside an apply {} block left over from the
docs-config.gradle refactor.

Assisted-by: Claude Code <Claude@Claude.ai>
Assisted-by: Claude Code <Claude@Claude.ai>
Copy link
Contributor

@jamesfredley jamesfredley left a comment

Choose a reason for hiding this comment

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

@jdaugherty jdaugherty merged commit 02110a7 into apache:7.0.x Mar 19, 2026
11 of 12 checks passed
@jdaugherty jdaugherty deleted the doc-combination branch March 19, 2026 04:36
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.

3 participants