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

Trivial: Corrected owner of DNS seeder #6038

Closed
wants to merge 1 commit into from
Closed

Trivial: Corrected owner of DNS seeder #6038

wants to merge 1 commit into from

Conversation

ayeowch
Copy link
Contributor

@ayeowch ayeowch commented Apr 20, 2015

No description provided.

@TheBlueMatt
Copy link
Contributor

utACK

@sandakersmann
Copy link
Contributor

ACK

@theuni
Copy link
Member

theuni commented Apr 20, 2015

ACK for trivial. @ayeowch for the future, it'd be great if you could submit these directly to trivial-next in https://github.com/theuni/bitcoin . The idea is to pile up small changes like this one over there, to avoid burning extra review cycles in the main repo. Every few weeks those changes are then submitted as a group in a single PR for inclusion in master.

@ayeowch
Copy link
Contributor Author

ayeowch commented Apr 21, 2015

@theuni Thanks for the note. Will do that in the future.

@laanwj
Copy link
Member

laanwj commented Apr 21, 2015

/me gives @fanquake a donkey hat, Not only you got sipa's name wrong, you here got the entire person wrong. Do your homework :)

... though this proves it's good to have the names there.

@laanwj laanwj added the Trivial label Apr 21, 2015
@cdecker
Copy link
Contributor

cdecker commented Apr 21, 2015

Ack, how did that happen in the first place? :-)

@sipa
Copy link
Member

sipa commented Apr 21, 2015

ACK

theuni added a commit to theuni/bitcoin that referenced this pull request Apr 23, 2015
@laanwj
Copy link
Member

laanwj commented Apr 23, 2015

Merged into the trivial branch, closing

@laanwj laanwj closed this Apr 23, 2015
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
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.

None yet

7 participants