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
add mapping for ssh property in build section #234
Conversation
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
loader/loader_test.go
Outdated
test: | ||
build: | ||
context: . | ||
ssh: ssh-agent |
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.
according to the spec proposal, should be default
types/types.go
Outdated
@@ -426,6 +427,9 @@ func (l Labels) Add(key, value string) Labels { | |||
return l | |||
} | |||
|
|||
// SSHConfig is a mapping type for SSH build config | |||
type SSHConfig map[string]string |
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 wonder SSHConfig
should be a []SSHKey
, the latter declared as struct { ID, Path }` (to match the spec)
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
ready to merge, just waiting for compose-spec/compose-spec#234 to be approved |
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
5ca58af
to
814bb73
Compare
Is this all good to merge now ? The spec has been approved ? |
This implement compose-spec/compose-spec#234