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.

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

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

Open
wants to merge 17 commits into
base: develop/0.7.0
Choose a base branch
from

Conversation

GuusLieben
Copy link
Member

@GuusLieben GuusLieben commented Oct 11, 2024

Type of Change

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

Description

Work in progress (primarily for CI). 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 added a commit that referenced this pull request Dec 24, 2024
@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant