Skip to content

fix: prevent Maven URL inheritance from appending module paths to repository links#302

Merged
AntonLem merged 2 commits into
mainfrom
copilot/update-pom-file-repositories
May 9, 2026
Merged

fix: prevent Maven URL inheritance from appending module paths to repository links#302
AntonLem merged 2 commits into
mainfrom
copilot/update-pom-file-repositories

Conversation

Copilot AI commented May 9, 2026

Copy link
Copy Markdown
Contributor
  • Identify the issue: parent pom.xml is not sorted/formatted correctly after the previous change (adding child.project.url.inherit.append.path attributes)
  • Run sortpom:sort to reformat pom.xml according to the project's sortpom configuration
  • Verified sortpom:verify passes (BUILD SUCCESS)
  • Verified inherited URLs still resolve correctly to https://github.com/lemon-ant/JHarmonizer in all child modules

@AntonLem
AntonLem marked this pull request as ready for review May 9, 2026 16:48
Copilot AI review requested due to automatic review settings May 9, 2026 16:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 prevents Maven’s default URL/SCM inheritance behavior from appending child module paths (e.g., /jharmonizer-core) to repository links, keeping all modules’ resolved project/SCM URLs pointing at the real GitHub repository root.

Changes:

  • Disabled child URL inheritance path appending via child.project.url.inherit.append.path="false" on the root <project>.
  • Disabled SCM connection/developerConnection/url path appending via the three child.scm.*.inherit.append.path="false" attributes on <scm>.

Agent-Logs-Url: https://github.com/lemon-ant/JHarmonizer/sessions/b46f36fa-dc21-41d9-80f5-8da5c5af971a

Co-authored-by: AntonLem <2189039+AntonLem@users.noreply.github.com>
Copilot AI requested a review from AntonLem May 9, 2026 17:02
@AntonLem
AntonLem merged commit 32fa052 into main May 9, 2026
1 check passed
@AntonLem
AntonLem deleted the copilot/update-pom-file-repositories branch May 9, 2026 17:10
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