-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[go] import command implemented #543
Conversation
releaseOut.Values = userValues | ||
releaseOut.Repository = repository | ||
|
||
// in order to set indent to 2 spaces we need to use a yaml encoder instead of yaml.Marshal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this was fixed in Yaml v3 in the later versions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It still seems to default to 4 spaces in my testing and I didn't see a way to change it in the library without opting for the encoder route. I can go look again.
if len(splitChartVersion) < 2 { | ||
return fmt.Errorf("could not parse chart version from %s - expected at least one hyphen between chart name and version", i.Chart) | ||
} | ||
splitPoint := len(splitChartVersion) - 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They made this annoying to get, didn't they?
Kudos, SonarCloud Quality Gate passed!
|
This PR fixes #526
Checklist
Description
What's the goal of this PR?
Added
reckoner import
functionality. I made the decision that thisoutput will always be in v2 schema format since we shouldn't support the
old style once the go version is released. That information is printed
as a warning when
import
is run.