Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding sections of pom requested by central repository #6

Merged

Conversation

alobodzki
Copy link
Contributor

No description provided.

pom.xml Outdated
</licenses>

<scm>
<connection>scm:git:git://github.com/netty/netty.git</connection>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this reference to a netty git repository intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Totally not.

@alobodzki alobodzki force-pushed the maven-repository-pom-settings branch 2 times, most recently from 2fcbbb0 to 7f65f2f Compare October 18, 2017 16:12
Copy link
Contributor

@massdosage massdosage left a comment

Choose a reason for hiding this comment

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

This mostly looks fine, I just have the one question about ssh in the scm url

pom.xml Outdated

<scm>
<connection>scm:git:git@github.com:HotelsDotCom/styx.git</connection>
<developerConnection>scm:git:ssh://git@github.com:HotelsDotCom/styx.git</developerConnection>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does this have "ssh" in it but the normal connection doesn't? I think this could just be the same "scm:git:git@github.com:HotelsDotCom/styx.git". This is what we have in WaggleDance which pushes artifacts to Sonatype.

@alobodzki alobodzki force-pushed the maven-repository-pom-settings branch from 7f65f2f to 03e95f9 Compare October 19, 2017 12:38
@mikkokar mikkokar merged commit 87ffd25 into ExpediaGroup:master Oct 19, 2017
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.

3 participants