Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

Docker app should only be used as a docker CLI plugin #667

Merged
merged 1 commit into from Oct 4, 2019

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Oct 4, 2019

- What I did
Removed support for docker-app-standalone and clarified installation doc

- How I did it
grep 'standalone' then rm

- How to verify it
make does not produce the standalone binary

- Description for the changelog
Docker app is only available as a docker CLI plugin

- A picture of a cute animal (not mandatory but encouraged)
image

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Copy link
Member

@rumpl rumpl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov
Copy link

codecov bot commented Oct 4, 2019

Codecov Report

Merging #667 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #667   +/-   ##
=======================================
  Coverage   72.07%   72.07%           
=======================================
  Files          52       52           
  Lines        2636     2636           
=======================================
  Hits         1900     1900           
  Misses        492      492           
  Partials      244      244

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0942d5c...41ce0fa. Read the comment docs.

@ndeloof ndeloof marked this pull request as ready for review October 4, 2019 08:36
@ndeloof ndeloof merged commit bcdf883 into docker:master Oct 4, 2019
@ndeloof ndeloof deleted the no-standalone branch October 4, 2019 08:37
ndeloof added a commit to ndeloof/app that referenced this pull request Oct 4, 2019
follow up for docker#667

Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants