-
Notifications
You must be signed in to change notification settings - Fork 314
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
Generate tequilapi json as build artifact #537
Conversation
interro
commented
Nov 13, 2018
•
edited
Loading
edited
- tequilapi.json will be added to dev release
d95c7b3
to
e16b421
Compare
e16b421
to
fc62f59
Compare
.travis.yml
Outdated
- script: | ||
- go get github.com/go-swagger/go-swagger/cmd/swagger/ | ||
- bin/swagger_generate | ||
- bin/s3 sync tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- bin/s3 sync tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts | |
- bin/s3 cp tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sync
should be applied for directories only, cp
for files.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed
.travis.yml
Outdated
- script: | ||
- go get github.com/go-swagger/go-swagger/cmd/swagger/ | ||
- bin/swagger_generate | ||
- bin/s3 cp tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- bin/s3 cp tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts | |
- bin/s3 cp tequilapi.json s3://travis-$TRAVIS_BUILD_NUMBER/build-artifacts/ |
and /
at the end, or it will create file /build-artifacts
instead of directory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
…MysteriumNetwork/node into tequilapi-build-artifact
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.