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.

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

tests for Daemon Listener #59 #88

Closed
wants to merge 3 commits into from
Closed

tests for Daemon Listener #59 #88

wants to merge 3 commits into from

Conversation

llSourcell
Copy link
Contributor

Daemon test and using IDfromPubKey instead of hardcoding in the public key.

@whyrusleeping
Copy link
Member

Looks like it needs a go-fmt run over it

@whyrusleeping
Copy link
Member

Hrm, it should. Your imports arent lined up and there are places where you have too many spaces (both of which would be fixed by go-fmt)

@whyrusleeping
Copy link
Member

alright, looks good to me, lets get a +1 from @perfmode

@btc
Copy link
Contributor

btc commented Sep 18, 2014

LGTM

@whyrusleeping
Copy link
Member

Merged!

@aschmahmann aschmahmann mentioned this pull request Sep 22, 2020
72 tasks
@aschmahmann aschmahmann mentioned this pull request Dec 1, 2021
80 tasks
@aschmahmann aschmahmann mentioned this pull request Dec 13, 2021
59 tasks
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
…/libp2p/go-libp2p-core-0.5.6

chore(deps): bump github.com/libp2p/go-libp2p-core from 0.5.0 to 0.5.6
laurentsenta pushed a commit to laurentsenta/kubo that referenced this pull request Feb 25, 2022
…dules/github.com/libp2p/go-libp2p-core-0.5.6

chore(deps): bump github.com/libp2p/go-libp2p-core from 0.5.0 to 0.5.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants