Skip to content
This repository was archived by the owner on Jul 16, 2021. It is now read-only.

show monocular version in ui#337

Merged
prydonius merged 3 commits into
helm:masterfrom
prydonius:316-show-version-in-homepage
Aug 17, 2017
Merged

show monocular version in ui#337
prydonius merged 3 commits into
helm:masterfrom
prydonius:316-show-version-in-homepage

Conversation

@prydonius
Copy link
Copy Markdown
Member

closes #316

Comment thread .travis.yml Outdated
- make -C src/api bootstrap
- make -C src/ui install
- |
if [[ -n "$TRAVIS_TAG" ]]; then
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

cc @sameersbn this will work as intended, right?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think you meant to write

make -C src/ui VERSION=$TRAVIS_TAG set-version

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

As an aside, if you want to git rid of the inline bash from travis, then you could replace the VERSION ?= dev assignment in the Makefile with.

ifeq "$(VERSION)" ""
	override VERSION = dev
endif

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

oops good catch, thanks

Comment thread .travis.yml Outdated
- make -C src/api bootstrap
- make -C src/ui install
- |
if [[ -n "$TRAVIS_TAG" ]]; then
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

As an aside, if you want to git rid of the inline bash from travis, then you could replace the VERSION ?= dev assignment in the Makefile with.

ifeq "$(VERSION)" ""
	override VERSION = dev
endif

@prydonius
Copy link
Copy Markdown
Member Author

@sameersbn PTAL!

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #337 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #337   +/-   ##
=======================================
  Coverage   88.41%   88.41%           
=======================================
  Files          17       17           
  Lines         820      820           
=======================================
  Hits          725      725           
  Misses         61       61           
  Partials       34       34

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 13f7c27...b1a1304. Read the comment docs.

@prydonius prydonius merged commit 9c9544a into helm:master Aug 17, 2017
@prydonius prydonius deleted the 316-show-version-in-homepage branch August 17, 2017 11:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show version number in the home page

3 participants