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

Update travis to always run repo-sync#287

Merged
prydonius merged 1 commit into
helm:masterfrom
prydonius:update-repo-sync-travis
Jun 23, 2017
Merged

Update travis to always run repo-sync#287
prydonius merged 1 commit into
helm:masterfrom
prydonius:update-repo-sync-travis

Conversation

@prydonius
Copy link
Copy Markdown
Member

This will ensure that updates to the chart source (indepdendent to
app/docker image updates) are published to the chart repository.

This commit also fixes a bug in the repo-sync script to ensure existing
versions are correctly skipped.

@sameersbn
Copy link
Copy Markdown

lgtm!

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jun 23, 2017

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #287   +/-   ##
======================================
  Coverage    90.4%   90.4%           
======================================
  Files          18      18           
  Lines         969     969           
======================================
  Hits          876     876           
  Misses         62      62           
  Partials       31      31

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 a836999...4aa6051. Read the comment docs.

This will ensure that updates to the chart source (indepdendent to
app/docker image updates) are published to the chart repository.

This commit also fixes a bug in the repo-sync script to ensure existing
versions are correctly skipped.
@prydonius prydonius force-pushed the update-repo-sync-travis branch from aa29cbc to 4aa6051 Compare June 23, 2017 10:04
@prydonius prydonius merged commit 333c66c into helm:master Jun 23, 2017
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.

3 participants