Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

All user to force install package with --force#744

Merged
bryanl merged 1 commit intoksonnet:masterfrom
bryanl:force-package-install
Jul 13, 2018
Merged

All user to force install package with --force#744
bryanl merged 1 commit intoksonnet:masterfrom
bryanl:force-package-install

Conversation

@bryanl
Copy link
Member

@bryanl bryanl commented Jul 12, 2018

This feature is added to allow two versions of a package to be
installed. (e.g. having multiple versions of a helm chart installed)

cc: @GuessWhoSamFoo

Signed-off-by: bryanl bryanliles@gmail.com

This feature is added to allow two versions of a package to be
installed. (e.g. having multiple versions of a helm chart installed)

Signed-off-by: bryanl <bryanliles@gmail.com>
@bryanl bryanl requested a review from a team July 12, 2018 23:43
@GuessWhoSamFoo GuessWhoSamFoo added this to the 0.12.0 milestone Jul 12, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1190

  • 10 of 10 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 71.301%

Totals Coverage Status
Change from base Build 1189: 0.01%
Covered Lines: 11853
Relevant Lines: 16624

💛 - Coveralls

if ok {
if ok && !force {
return nil, errors.Errorf("package '%s/%s@%s' already exists.",
libRef.Registry, libRef.Name, libRef.Version)
Copy link
Collaborator

Choose a reason for hiding this comment

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

libRef.Version returns "" rather than SemVer; might be confusing later on

Copy link
Collaborator

@GuessWhoSamFoo GuessWhoSamFoo left a comment

Choose a reason for hiding this comment

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

lgtm

@bryanl bryanl merged commit d225af8 into ksonnet:master Jul 13, 2018
@bryanl bryanl deleted the force-package-install branch July 13, 2018 00:05
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