8000 Make error messages on ring merge clearer by bboreham · Pull Request #1957 · weaveworks/weave · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

Make error messages on ring merge clearer #1957

Merged
merged 1 commit into from
Feb 8, 2016
Merged

Conversation

bboreham
Copy link
Contributor
@bboreham bboreham commented Feb 5, 2016

Addresses one of the points raised in #1949

@rade
Copy link
Member
rade commented Feb 5, 2016

perhaps ErrNewerVersion should be clearer too.

@bboreham bboreham force-pushed the issues/1949-ring-errors branch from 94cef5d to 034da37 Compare February 8, 2016 10:13
@bboreham
Copy link
Contributor Author
bboreham commented Feb 8, 2016

I changed ErrNewerVersion

@awh awh self-assigned this Feb 8, 2016
awh added a commit that referenced this pull request Feb 8, 2016
Make error messages on ring merge clearer
@awh awh merged commit 6d1e33b into master Feb 8, 2016
@awh awh deleted the issues/1949-ring-errors branch February 8, 2016 13:53
@awh awh added this to the 1.5.0 milestone Feb 8, 2016
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
0