Skip to content
This repository was archived by the owner on Aug 29, 2023. It is now read-only.

fix for issue #250#251

Open
fitsegreat wants to merge 2 commits into
weaveworks:masterfrom
fitsegreat:master
Open

fix for issue #250#251
fitsegreat wants to merge 2 commits into
weaveworks:masterfrom
fitsegreat:master

Conversation

@fitsegreat
Copy link
Copy Markdown

Adjusted the proper return value

Copy link
Copy Markdown
Contributor

@bboreham bboreham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, but we cannot accept this PR as it stands.

Comment thread pkg/cluster/machine.go
}

return
return b
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The statement before means the same thing, since b is specified as a named return value.
In other words, you did not make any change to the meaning of the code.

Comment thread .goreleaser.yml
brews:
-
github:
tap:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please explain more about why this change is needed.
Does it fix an issue?

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.

2 participants