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

require env to exist before making it current#818

Merged
bryanl merged 1 commit intoksonnet:masterfrom
bryanl:746-ks-env-current-requires-valid-env
Aug 6, 2018
Merged

require env to exist before making it current#818
bryanl merged 1 commit intoksonnet:masterfrom
bryanl:746-ks-env-current-requires-valid-env

Conversation

@bryanl
Copy link
Member

@bryanl bryanl commented Aug 6, 2018

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

Signed-off-by: bryanl <bryanliles@gmail.com>
@bryanl bryanl requested a review from a team August 6, 2018 14:52
@bryanl bryanl self-assigned this Aug 6, 2018
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1271

  • 9 of 11 (81.82%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 70.989%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/app/base_app.go 9 11 81.82%
Totals Coverage Status
Change from base Build 1270: 0.005%
Covered Lines: 12054
Relevant Lines: 16980

💛 - Coveralls

@bryanl bryanl merged commit af24580 into ksonnet:master Aug 6, 2018
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.

2 participants