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

Fix sourcePath for imported json on root module#844

Merged
GuessWhoSamFoo merged 1 commit intoksonnet:masterfrom
GuessWhoSamFoo:import-json
Sep 4, 2018
Merged

Fix sourcePath for imported json on root module#844
GuessWhoSamFoo merged 1 commit intoksonnet:masterfrom
GuessWhoSamFoo:import-json

Conversation

@GuessWhoSamFoo
Copy link
Collaborator

This PR removes the extra / added because the default module is / since 0.11

Added test to hit the code path on https://github.com/ksonnet/ksonnet/blob/master/pkg/actions/import.go#L203

Signed-off-by: GuessWhoSamFoo sfoohei@gmail.com

@GuessWhoSamFoo GuessWhoSamFoo requested a review from a team August 28, 2018 14:25
@coveralls
Copy link

coveralls commented Aug 28, 2018

Pull Request Test Coverage Report for Build 1309

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 70.64%

Totals Coverage Status
Change from base Build 1307: 0.1%
Covered Lines: 12316
Relevant Lines: 17435

💛 - Coveralls

Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>
@GuessWhoSamFoo GuessWhoSamFoo merged commit 4769eef into ksonnet:master Sep 4, 2018
@GuessWhoSamFoo GuessWhoSamFoo deleted the import-json branch September 4, 2018 21:40
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