-
Notifications
You must be signed in to change notification settings - Fork 457
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
allow iptables package to be updated #583
Conversation
|
Thanks @cristifalcas! This looks like a good addition. Can you please squash the two commits into one? |
|
Hi again @cristifalcas, would you mind squashing this down? |
fb11dc4
to
22ec6d8
Compare
|
sorry for the delay |
|
@cristifalcas Thanks for the work that you have put in, there are a number tests failing, could you please have a look. |
|
Fixed |
|
Thanks, can you please squash. the code looks good to me. |
|
I have no idea how to remove the "Merge remote-tracking branch" commit. Is it ok if I close this PR and open a new one? |
|
if you are on the CLI: it will look something like pick 690cb20 Adding bug workaround in tests for Debian 8 change it to pick 690cb20 Adding bug workaround in tests for Debian 8 This will squash that commit. Then re-push again |
|
or create a new pr 👍 |
4694697
to
85a0b60
Compare
cfbdfcc
to
9378559
Compare
|
done |
|
great work !!! |
allow iptables package to be updated
Add a new parameter in order to allow the iptables package to be updated