Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Generalize peerid check #1692

Merged
merged 6 commits into from
Sep 13, 2015
Merged

Generalize peerid check #1692

merged 6 commits into from
Sep 13, 2015

Conversation

chriscool
Copy link
Contributor

This is a followup from PR #1625.

@chriscool
sharness: generalize test_check_peerid()
5c118a2 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool
t0060: simplify peerid check
0aa5936 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool
t0110: add peerid check
9563cb2 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool
t0062: add peerid check
bec8224 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool
t0100: add peerid check
e22cbce 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@chriscool
test-lib: improve test_check_peerid
6e59b39 
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@jbenet jbenet added the status/in-progress In progress label Sep 12, 2015
@chriscool
Copy link
Contributor Author

By the way thanks @rht for suggesting this in PR #1625.

@jbenet
Copy link
Member

jbenet commented Sep 13, 2015

LGTM thanks @chriscool

jbenet added a commit that referenced this pull request Sep 13, 2015
@jbenet
Merge pull request #1692 from ipfs/generalize-peerid-check
a8dc708 
Generalize peerid check
@jbenet jbenet merged commit a8dc708 into master Sep 13, 2015
@jbenet jbenet removed the status/in-progress In progress label Sep 13, 2015
@jbenet jbenet deleted the generalize-peerid-check branch September 13, 2015 04:42
@chriscool chriscool mentioned this pull request Sep 14, 2015
Closed
41 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chriscool @jbenet