-
Notifications
You must be signed in to change notification settings - Fork 158
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
Automate release #252
Automate release #252
Conversation
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
I have done release 1.12 and 1.13 locally to verify if sed commands work |
I will remove those commits once this pr is approved |
fi | ||
} | ||
|
||
if [[ "$TRAVIS_TAG" =~ ^release-[[:digit:]]+\.[[:digit:]]+\.[[:digit:]]+?$ ]]; then |
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.
Space looks like typo, can say we are on release-..* tag
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.
which space? I think condition works well.
|
||
set -o errexit -o nounset -o pipefail | ||
|
||
safe_checkout_master() { |
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.
Is this code copied from somewhere? Feels familiar, we could add attribution comment.
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.
It's from java client, It is fairly simple code
travis/release.sh
Outdated
|
||
if [[ "$TRAVIS_TAG" =~ ^release-[[:digit:]]+\.[[:digit:]]+\.[[:digit:]]+?$ ]]; then | ||
echo "We are on release- tag" | ||
echo "Creating new documentation version" |
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.
Suggestion: move after L26 and include version in the message
.travis.yml
Outdated
before_install: | ||
- git config user.name "jaegertracingci" | ||
- git config user.email "jaegertracingci+jaeger-tracing@googlegroups.com" | ||
# setup https authentication credentials, used by ./gradlew publish |
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.
Comment not applicable?
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
7175b32
to
9e272a5
Compare
Resolves #249