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

docker-machine: 0.10.0 -> 0.12.0 #26875

Closed
wants to merge 1 commit into from
Closed

docker-machine: 0.10.0 -> 0.12.0 #26875

wants to merge 1 commit into from

Conversation

1755
Copy link

@1755 1755 commented Jun 26, 2017

Motivation for this change
Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@joachifm
Copy link
Contributor

cc @tailhook @offlinehacker

@Mic92 Mic92 changed the title Update docker-machine version docker-machine: 0.10.0 -> 0.12.0 Jun 27, 2017
@joachifm
Copy link
Contributor

I take it this received no testing?

@1755
Copy link
Author

1755 commented Jun 28, 2017

@joachifm Unfortunately no. I just​ update the version and sha256 hash. It was needed for my task. I just decided to share this quickfix

@grahamc
Copy link
Member

grahamc commented Jul 16, 2017

Thank you, @1755! Can you update your commit message to say "docker-machine: 0.10.0 -> 0.12.0"? If not, I can do it for you. Otherwise, it looks good to merge.

@1755
Copy link
Author

1755 commented Jul 18, 2017

@grahamc I have not cloned repo. Can you do it please?

@Mic92
Copy link
Member

Mic92 commented Jul 18, 2017

@grahamc you can also do that, when using squasing instead of merging.

@grahamc
Copy link
Member

grahamc commented Jul 26, 2017

Applied in 6c8871f

@grahamc grahamc closed this Jul 26, 2017
@grahamc
Copy link
Member

grahamc commented Jul 26, 2017

Thank you! I'm sorry this took so long :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants