Skip to content

fix/mavenurl#124

Open
wangyb-A wants to merge 1 commit intomainfrom
fix/mavenurl
Open

fix/mavenurl#124
wangyb-A wants to merge 1 commit intomainfrom
fix/mavenurl

Conversation

@wangyb-A
Copy link
Contributor

@wangyb-A wangyb-A commented Feb 26, 2026

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

Description

Fix maven url in pom

  • We only set urls in parent POM before, flatten-pom-plugin will automatically append the artifact name in the URL, causing the url on maven to break.
  • I added and field in child pom so that it will not be overriden by the plugin.
  • This fix will take effect when next release is published to maven.
  • For the url used in section, please refer to: https://maven.apache.org/pom.html#scm

Demo/Screenshots

N/A

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

mvn clean install

Manually verified that in the .flattened-pom.xml, we are using

    <url>https://github.com/aws/aws-durable-execution-sdk-java</url>

    <scm>
        <connection>scm:git:https://github.com/aws/aws-durable-execution-sdk-java.git</connection>
        <developerConnection>scm:git:https://github.com/aws/aws-durable-execution-sdk-java.git</developerConnection>
        <url>https://github.com/aws/aws-durable-execution-sdk-java</url>
    </scm>

Unit Tests

N/A

Integration Tests

N/A

Examples

N/A

@wangyb-A wangyb-A marked this pull request as ready for review February 26, 2026 18:50
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