-
Notifications
You must be signed in to change notification settings - Fork 38
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
Release adjustments #176
Release adjustments #176
Conversation
Signed-off-by: wenqi <Wenqi.Mou@emc.com>
Signed-off-by: wenqimou <452787782@qq.com>
@adrianmo Could you please take a look at the change in Travis? Thank you very much! |
Makefile
Outdated
@@ -57,7 +57,9 @@ login: | |||
|
|||
push: build login | |||
docker push $(REPO):$(VERSION) | |||
docker push $(REPO):latest | |||
if [[ ${TRAVIS_TAG} =~ ^[0-9]+(\.[0-9]+)+(\.[0-9]+)$ ]]; 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.
You don't need to create capture groups. Check out the regex pattern we are already using in the code: https://github.com/pravega/pravega-operator/blob/master/pkg/util/pravegacluster.go#L29
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.
Got it, thanks!
README.md
Outdated
@@ -1,6 +1,6 @@ | |||
# Pravega Operator | |||
|
|||
[![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) [![GoDoc](https://godoc.org/github.com/pravega/pravega-operator?status.svg)](https://godoc.org/github.com/pravega/pravega-operator) [![Build Status](https://travis-ci.org/pravega/pravega-operator.svg?branch=master)](https://travis-ci.org/pravega/pravega-operator) [![Go Report](https://goreportcard.com/badge/github.com/pravega/pravega-operator)](https://goreportcard.com/report/github.com/pravega/pravega-operator) | |||
[![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) [![GoDoc](https://godoc.org/github.com/pravega/pravega-operator?status.svg)](https://godoc.org/github.com/pravega/pravega-operator) [![Build Status](https://travis-ci.org/pravega/pravega-operator.svg?branch=master)](https://travis-ci.org/pravega/pravega-operator) [![Go Report](https://goreportcard.com/badge/github.com/pravega/pravega-operator)](https://goreportcard.com/report/github.com/pravega/pravega-operator)[![Version](https://img.shields.io/badge/release-0.4.0-green.svg)](https://github.com/pravega/pravega-operator/releases) |
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 it possible to add a dynamic version badge that is automatically updated when releases are published?
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.
Fixed! Thanks
Signed-off-by: wenqimou <452787782@qq.com>
Signed-off-by: wenqimou <452787782@qq.com>
Signed-off-by: wenqimou <452787782@qq.com>
Change log description
latest
Docker tag when release candidates are published.Signed-off-by: wenqi Wenqi.Mou@emc.com