Skip to content

Use separate POM files for core and monkey artifacts - #12

Closed
toaster0123 wants to merge 1 commit into
Electrostat-Lab:masterfrom
toaster0123:audit/separate-pom-paths-20261013
Closed

toaster0123 wants to merge 1 commit into
Electrostat-Lab:masterfrom
toaster0123:audit/separate-pom-paths-20261013

Conversation

@toaster0123

@toaster0123 toaster0123 commented Oct 13, 2026 •

Copy link
Copy Markdown

Problem

Both publications write their generated metadata to jector.pom. Generating the monkey POM overwrites the core POM before the publishing step.

Change

Use jector-monkey.pom for the monkey artifact and add a regression check for distinct, correctly named POM paths.

Validation

  • python3 helper-scripts/tests/check-pom-paths.py: fails on upstream bdca41a, passes with this fix
  • sh -n helper-scripts/project-impl/variables.sh and git diff --check: pass

Signing and artifact publication were not performed. Existing published metadata would require a new release.

This automated PR is part of the jMonkeyEngine community library quality check.

@codacy-production

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 medium

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
ErrorProne 1 medium

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@toaster0123
toaster0123 marked this pull request as ready for review October 13, 2026 13:43
@toaster0123

Copy link
Copy Markdown
Author

Superseded by #13, which contains the same fix submitted from jaime-jmebot. Please review the replacement PR: #13

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.

1 participant