Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Re-organize platform POMs for building, staging, assembly, and tooling support #1114

Merged
merged 20 commits into from
Feb 11, 2025
Merged

Re-organize platform POMs for building, staging, assembly, and tooling support #1114

merged 20 commits into from
Feb 11, 2025

Conversation

GuusLieben
Copy link
Member

@GuusLieben GuusLieben commented Oct 11, 2024 •
edited
Loading

Type of Change

  • Chore (changes to the build process or auxiliary tools)

Description

Re-organizes our POM structure to allow modules to:

  • be published more cleanly (without the project root POM being published)
  • have dedicated POMs for staging, build configuration, style configurations, and dependency updates
  • better utilize the assembly module for documentation and test aggregations
  • generally clean up the structure of our Maven configurations

Checklist

  • I have performed a self-review of my own changes
  • New and existing unit tests pass locally with my changes
  • I have added documentation that describes my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have rebased my branch on top of the latest develop branch

@GuusLieben GuusLieben added this to the 0.7.0 milestone Oct 11, 2024
@GuusLieben GuusLieben self-assigned this Oct 11, 2024
@GuusLieben GuusLieben force-pushed the draft/platform-poms branch 4 times, most recently from 4232488 to 3b53861 Compare December 20, 2024 16:28
@GuusLieben
#1114 Generate manifest for artifacts
4cc4884
@GuusLieben
#1114 Re-organize platform POMs for building, staging, assembly, and …
b7530dc 
…tooling support

# Conflicts:
#	pom.xml
@GuusLieben
#1114 Remove outdated comment on assembly parent POM
c1c8408
@GuusLieben
#1114 Ensure assembly paths are correctly resolved within multimodule…
0f90a4d 
… context
@GuusLieben
#1114 Move test fixtures to separate Maven modules
d23438b
@GuusLieben
#1114 Organize test fixtures, add documentation where needed
6707df4
@GuusLieben
#1114 Update CI configurations to use Maven phases instead of direct …
dc89b8a 
…plugin goals
@GuusLieben
#1114 Update license headers for modified components
c7680e9
@GuusLieben
#1114 Add execution for version updates
3136f54
@GuusLieben
#1114 Extract version updates to platform module
032fb2a
@GuusLieben
#1114 Bump BOM and plugin dependency versions
8ca1670
@GuusLieben
#1114 Move Antora generation to install phase
5d36da5
GuusLieben added a commit that referenced this pull request Dec 24, 2024
@GuusLieben
#1114 Move Antora generation to Assembly
9397667
@GuusLieben GuusLieben changed the title Draft: Re-organize platform POMs for building, staging, assembly, and tooling support Re-organize platform POMs for building, staging, assembly, and tooling support Dec 25, 2024
@GuusLieben GuusLieben marked this pull request as ready for review December 25, 2024 14:27
@GuusLieben
Copy link
Member Author

Requires publishing test (deploy and sign verification) before merging. After publishing has been confirmed to be functional, this PR can be merged.

@GuusLieben GuusLieben added Status: needs triage Waiting for triage to be completed and removed type: chore labels Jan 17, 2025
@GuusLieben GuusLieben removed the Status: needs triage Waiting for triage to be completed label Feb 11, 2025
@GuusLieben GuusLieben merged commit d7961c2 into develop/0.7.0 Feb 11, 2025
9 checks passed
@GuusLieben GuusLieben deleted the draft/platform-poms branch February 11, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@GuusLieben GuusLieben

Labels
None yet
Milestone
0.7.0
Development

Successfully merging this pull request may close these issues.
1 participant
@GuusLieben