Skip to content

Review fixes for Doc Combination PR#1212

Closed
jamesfredley wants to merge 5 commits into7.0.xfrom
fix/doc-combination-review
Closed

Review fixes for Doc Combination PR#1212
jamesfredley wants to merge 5 commits into7.0.xfrom
fix/doc-combination-review

Conversation

@jamesfredley
Copy link
Contributor

Summary

Review fixes for #1211. Merge this into doc-combination before merging that PR.

  • Update stale Spring Security API links - index.adoc header links were hardcoded to Spring Security 4.0.x and Spring Framework 4.1.x. Now uses AsciiDoc attributes populated from the BOM-resolved version in docs/build.gradle.
  • Remove duplicate reproducible-config.gradle applies - All 7 plugin docs/build.gradle files applied it twice: once at the top level and again inside an apply {} block left over from the docs-config.gradle refactor.
  • Replace em dashes with hyphens in whatsNew.adoc

jdaugherty and others added 3 commits March 14, 2026 17:59
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>
@github-actions github-actions bot added the bug label Mar 18, 2026
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>
@jamesfredley jamesfredley force-pushed the fix/doc-combination-review branch from f392295 to 4c120c7 Compare March 19, 2026 00:02
@jdaugherty
Copy link
Contributor

Merged into my branch

@jdaugherty jdaugherty closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants