Skip to content

Fix AMP-31215: correct liquibase artifactId in api-gateway pom - #103

Merged
jdeanquin-dg merged 1 commit into
amp-main-stablefrom
fix/AMP-31215/api-gateway-liquibase-artifact-id
Sep 18, 2026
Merged

jdeanquin-dg merged 1 commit into
amp-main-stablefrom
fix/AMP-31215/api-gateway-liquibase-artifact-id

Conversation

@jdeanquin-dg

Copy link
Copy Markdown

Summary

  • CI build for gateway was failing on mvn dependency:go-offline with Could not find artifact org.liquibase:liquibase:jar:4.29.0 in central.
  • The AMP-31215 cleanup (removing the proprietary liquibase-commercial dependency, AMP-31215 : Remove proprietary liquibase-commercial dependency pulled into api-commons/commons #102) replaced it with org.liquibase:liquibase, but Maven Central only publishes this library as liquibase-core — which is what the root pom.xml already correctly uses.
  • Fixes api-gateway/pom.xml to use liquibase-core, matching the root pom.

Test plan

  • mvn dependency:resolve in api-gateway/ resolves org.liquibase:liquibase-core:4.29.0 cleanly
  • Re-run Build and push AMP Dashboard to registry workflow to confirm gateway service builds

🤖 Generated with Claude Code

The AMP-31215 cleanup replaced the proprietary liquibase-commercial
dependency with org.liquibase:liquibase, which does not exist on
Maven Central (the artifact is published as liquibase-core, matching
the root pom.xml). CI failed on `mvn dependency:go-offline` because
the artifact could not be found.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 18, 2026 19:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The dependency coordinate is corrected with no unresolved review issues.

Pull request overview

Corrects the API Gateway’s Liquibase Maven dependency to use the artifact published by Maven Central.

Changes:

  • Replaces liquibase with liquibase-core version 4.29.0.
  • Aligns the dependency with the root POM.
File summaries
File Description
api-gateway/pom.xml Corrects the Liquibase artifact ID.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@jdeanquin-dg
jdeanquin-dg merged commit b4cba05 into amp-main-stable Sep 18, 2026
1 check passed
@jdeanquin-dg
jdeanquin-dg deleted the fix/AMP-31215/api-gateway-liquibase-artifact-id branch September 18, 2026 19:16
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