-
Notifications
You must be signed in to change notification settings - Fork 42
go-modules: fix build, module name and local imports #12
go-modules: fix build, module name and local imports #12
Conversation
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.
Would prefer two separated PR as this addresses two unrelated issues. But if others agree in this PR as is, I won't block it for merging.
updated OP to include the reason for the rename. |
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.
There's a PR out for the DeleteNode fix
I also swear I thought I modified all the imports but git don't lie!
Thanks for this fix, just undo the Delete changes and it's good to go
- update module to github.com/kubernetes-sigs/cluster-api-provider-docker - modify github.com/chuckha/cluster-api-provider-docker imports to github.com/kubernetes-sigs/cluster-api-provider-docker/kind/actions run 'go mod tidy'
bb1c2ec
to
d357ca2
Compare
updated to exclude the Delete* rename. |
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: chuckha, neolit123 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
fix the build in current master:
github.com/kubernetes-sigs/cluster-api-provider-docker/kind/actions
run 'go mod tidy'
the existing functions don't have Cluster in the name:
DeleteControlPlane, DeleteWorker
/kind bug
/priority backlog
/assign @chuckha