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

Support ipv6, dns-*, source and more hooks #10

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

t-tran
Copy link

@t-tran t-tran commented Apr 29, 2016

gateway is not always an IP, it can also be something like: 12.34.56.78 metric 10

@nMustaki
Copy link
Contributor

@t-tran I maintain a fork of debinterface, would you mind to open a PR there too so can I can merge them in ? Thanks !

@t-tran
Copy link
Author

t-tran commented Nov 13, 2016

@nMustaki which branch do you want me to create a PR against? I tried both of your branches and it gave me Can’t automatically merge warning.

@nMustaki
Copy link
Contributor

devel please, and I'll fix the conflicts if you don't have the time to do it.

Thanks again !

@nMustaki
Copy link
Contributor

Hi, could you open your pr on my devel branch, I'd like to make a release very soon ! I can handle the conflicts if that's blocking you😉

@t-tran
Copy link
Author

t-tran commented Nov 27, 2016

@nMustaki I'm sorry, I'm puzzled. I thought I already created a PR on your devel branch, but I can't find it there now.

@nMustaki
Copy link
Contributor

nMustaki commented Nov 27, 2016 via email

@nMustaki
Copy link
Contributor

nMustaki commented Feb 1, 2017

Hi @t-tran , still waiting for your pull request on my fork, so I can merge it :-)

@t-tran
Copy link
Author

t-tran commented Feb 16, 2017

@nMustaki just did it, feel free to cherrypick anything from the PR.

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.

2 participants