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

Bump go-jsonnet version#597

Merged
bryanl merged 2 commits intoksonnet:masterfrom
stang:bump/go-jsonnet
Jul 1, 2018
Merged

Bump go-jsonnet version#597
bryanl merged 2 commits intoksonnet:masterfrom
stang:bump/go-jsonnet

Conversation

@stang
Copy link

@stang stang commented Jun 7, 2018

Bumping go-jsonnet to the current version so we can benefit of the massive performance improvement shipped in google/go-jsonnet#225.

@shomron
Copy link
Collaborator

shomron commented Jun 7, 2018

@stang thanks for the PR!

We prefer to include such upstream changes as part of a firm release - we've reached out to the jsonnet maintainers to find out when that can be expected.

@bryanl
Copy link
Member

bryanl commented Jun 28, 2018

@stang Would you mind updating this to the current master of jsonnet and rebasing off of our master? Also, could you review our DCO process and sign off your commit? https://github.com/ksonnet/ksonnet/blob/master/CONTRIBUTING.md

stang added 2 commits July 1, 2018 08:35
Signed-off-by: Stephane Tang <hi@stang.sh>
Signed-off-by: Stephane Tang <hi@stang.sh>
@stang stang force-pushed the bump/go-jsonnet branch from e7ec4f3 to 0777fc4 Compare July 1, 2018 08:28
@coveralls
Copy link

Pull Request Test Coverage Report for Build 1093

  • 0 of 5 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 71.169%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/util/jsonnet/import.go 0 5 0.0%
Totals Coverage Status
Change from base Build 1092: 0.0%
Covered Lines: 11002
Relevant Lines: 15459

💛 - Coveralls

@stang
Copy link
Author

stang commented Jul 1, 2018

Here we go @bryanl :)

@bryanl bryanl merged commit 604cf78 into ksonnet:master Jul 1, 2018
@stang stang deleted the bump/go-jsonnet branch July 2, 2018 09:08
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.

4 participants