Skip to content

Fix to publish Snapshots to jfrog #199

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

Merged
merged 1 commit into from
Nov 28, 2016

Conversation

NiteshKant
Copy link
Contributor

Problem

Currently travis is publishing 0.2.3-SNAPSHOT artifacts for 0.5.x branch.

Modifications

I am not 100% sure if this is the correct way to make travis publish the correct version, but it is a way!
Modified buildViaTravis.sh to override the version, just for snapshot builds.

Result

Correct version number for snapshot.

#### Problem

Currently travis is publishing 0.2.3-SNAPSHOT artifacts for 0.5.x branch.

#### Modifications

I am not 100% sure if this is the correct way to make travis publish the correct version, but it is a way!
Modified `buildViaTravis.sh` to override the version, just for snapshot builds.

#### Result

Correct version number for snapshot.
@NiteshKant
Copy link
Contributor Author

FYI @2m @ktoso @drewhk

@2m
Copy link

2m commented Nov 21, 2016

So this fixes the version number. But does this also mean that snapshots are deployed after every commit to JFrog?

@NiteshKant
Copy link
Contributor Author

@2m the snapshots are built and published via travis today. The only issue is that it is publishing them with the wrong version. This change (hopefully) fixes the version.

Copy link
Member

@stevegury stevegury left a comment

Choose a reason for hiding this comment

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

LGTM

@stevegury stevegury merged commit 63d4a26 into rsocket:0.5.x Nov 28, 2016
@NiteshKant
Copy link
Contributor Author

FYI: @2m the snapshots are published to JFrog now with 0.5.0-SNAPSHOT name.

NiteshKant added a commit that referenced this pull request Dec 8, 2016
#### Problem

Currently travis is publishing 0.2.3-SNAPSHOT artifacts for 0.5.x branch.

#### Modifications

I am not 100% sure if this is the correct way to make travis publish the correct version, but it is a way!
Modified `buildViaTravis.sh` to override the version, just for snapshot builds.

#### Result

Correct version number for snapshot.
NiteshKant added a commit to NiteshKant/reactivesocket-java that referenced this pull request Dec 14, 2016
Currently travis is publishing 0.2.3-SNAPSHOT artifacts for 0.5.x branch.

I am not 100% sure if this is the correct way to make travis publish the correct version, but it is a way!
Modified `buildViaTravis.sh` to override the version, just for snapshot builds.

Correct version number for snapshot.
@NiteshKant NiteshKant deleted the 0.5.x-snap-ver branch December 15, 2016 19:44
ilayaperumalg pushed a commit to ilayaperumalg/rsocket-java that referenced this pull request Dec 26, 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