Skip to content

Fix/maven central portal migration#765

Merged
Periecle merged 3 commits into
developfrom
fix/maven-central-portal-migration
Jun 29, 2026
Merged

Fix/maven central portal migration#765
Periecle merged 3 commits into
developfrom
fix/maven-central-portal-migration

Conversation

@Periecle

@Periecle Periecle commented Jun 29, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Chores
    • Added explicit project names across multiple modules and test artifacts.
    • Improved module identification in build metadata, making the project structure easier to understand in tooling and reports.

Roman Kvasnytskyi added 2 commits June 29, 2026 11:55
feign-reactor-parent forced maven-gpg-plugin to sign with gpg.keyname
3EEF24C7 and set gpg.defaultKeyring=false. Under maven-gpg-plugin 3.2.5
that combination failed to resolve the key in CI ("gpg: skipped
3EEF24C7: No secret key"), even though the org GPG_PRIVATE_KEY (which is
that key) was imported by setup-java -- the root module, which sets no
keyname, signed fine with the default key.

- Drop the hardcoded gpg.keyname/defaultKeyring (plus deprecated
  useagent/lockMode) so every module signs with the default imported
  key, matching the root module that already worked.
- Bump maven-gpg-plugin 3.2.5 -> 3.2.8 to align with the proven
  testcontainers-spring-boot release setup.
The Central Publisher Portal rejected the 5.0.0 deployment with
"Project name is missing" for every module. The Portal requires a
<name> in each published POM, and Maven does not inherit <name> from
the parent, so the 18 modules without one failed validation (only
feign-reactor-core, feign-reactor-test and the root declared a name).

Add <name>{artifactId}</name> to each remaining module, matching the
testcontainers-spring-boot convention that already publishes to the
Portal successfully.
@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ab8dfcf3-847c-4d90-890f-634393a05f5f

📥 Commits

Reviewing files that changed from the base of the PR and between e0fe272 and b2ed209.

📒 Files selected for processing (18)
  • feign-reactor-benchmarks/pom.xml
  • feign-reactor-bom/pom.xml
  • feign-reactor-cloud/pom.xml
  • feign-reactor-java11/pom.xml
  • feign-reactor-jetty/pom.xml
  • feign-reactor-parent/pom.xml
  • feign-reactor-rx2/pom.xml
  • feign-reactor-rx3/pom.xml
  • feign-reactor-spring-cloud-starter/pom.xml
  • feign-reactor-spring-configuration/pom.xml
  • feign-reactor-test/feign-reactor-server-configurations/pom.xml
  • feign-reactor-test/feign-reactor-spring-configuration-cloud2-test/pom.xml
  • feign-reactor-test/feign-reactor-spring-configuration-test/pom.xml
  • feign-reactor-test/feign-reactor-spring-mvc-test/pom.xml
  • feign-reactor-webclient-apache-client5/pom.xml
  • feign-reactor-webclient-core/pom.xml
  • feign-reactor-webclient-jetty/pom.xml
  • feign-reactor-webclient/pom.xml

📝 Walkthrough

Walkthrough

Adds a <name> element matching each module's artifact ID to 18 Maven POM files across the project. No dependencies, build configuration, or logic were changed.

Changes

Add Maven name elements to module POMs

Layer / File(s) Summary
Add <name> to all module POMs
feign-reactor-parent/pom.xml, feign-reactor-benchmarks/pom.xml, feign-reactor-bom/pom.xml, feign-reactor-cloud/pom.xml, feign-reactor-java11/pom.xml, feign-reactor-jetty/pom.xml, feign-reactor-rx2/pom.xml, feign-reactor-rx3/pom.xml, feign-reactor-spring-cloud-starter/pom.xml, feign-reactor-spring-configuration/pom.xml, feign-reactor-test/*/pom.xml, feign-reactor-webclient*/pom.xml
Adds a <name> element equal to the artifact ID in each of the 18 module POMs.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A name for each module, now proudly declared,
No longer anonymous, each POM repaired.
The rabbit hops by with a stamp and a nod—
Identity matters, even in prod! 🐇

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/maven-central-portal-migration

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Periecle
Periecle merged commit 6f04efb into develop Jun 29, 2026
4 of 5 checks passed
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