Skip to content

chore: clean up POM after class-pruner moved to its own repo#1210

Merged
chrjohn merged 2 commits intomasterfrom
copilot/cleanup-pom-file
May 5, 2026
Merged

chore: clean up POM after class-pruner moved to its own repo#1210
chrjohn merged 2 commits intomasterfrom
copilot/cleanup-pom-file

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 4, 2026

Now that quickfixj-class-pruner-maven-plugin lives at https://github.com/quickfix-j/quickfixj-class-pruner-maven-plugin, the source directory and all Maven artefacts that existed solely to support building it locally can be removed.

Changes

  • Delete quickfixj-class-pruner-maven-plugin/ — entire directory (Java source, tests, test resources, GitHub Actions workflow)
  • Root pom.xml<dependencyManagement> — remove 8 entries that were only consumed by the deleted module: maven-shared-utils, file-management, maven-core, maven-artifact, maven-compat, maven-plugin-testing-harness, plexus-utils, plexus-xml; plus their 4 version properties (maven-shared-utils.version, file-management.version, maven-plugin-testing-harness.version, maven-libs-version)
  • Root pom.xml<pluginManagement> — remove maven-invoker-plugin (was only used to run the class-pruner's integration tests)
  • quickfixj-distribution/pom.xml — remove quickfixj-class-pruner-maven-plugin dependency; it is an independent project now and should not be bundled in the distribution
  • quickfixj-messages-fix*/pom.xml — remove redundant <version>${class-pruner-plugin.version}</version> from each plugin invocation; the version is already pinned in root <pluginManagement>
  • quickfixj-messages/readme.md — replace broken relative link to the deleted readme.md with a link to the external GitHub repo

@chrjohn chrjohn added this to the QFJ 3.0.1 milestone May 4, 2026
@chrjohn chrjohn changed the title chore: clean up POM after class-pruner moved to its own repo chore: clean up POM after class-pruner moved to its own repo May 4, 2026
@chrjohn chrjohn marked this pull request as ready for review May 5, 2026 08:40
@chrjohn chrjohn merged commit e728739 into master May 5, 2026
20 of 21 checks passed
@chrjohn chrjohn deleted the copilot/cleanup-pom-file branch May 5, 2026 08:41
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.

2 participants