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

Mark docker app plugin as experimental #537

Merged
merged 1 commit into from
May 24, 2019

Conversation

silvin-lubecki
Copy link
Contributor

@silvin-lubecki silvin-lubecki commented May 22, 2019

- What I did

  • Mark the app plugin as an experimental plugin.

- How to verify it

  1. Use this docker/cli
  2. Build the docker app plugin
  3. Move it to your ~/.docker/cli-plugins directory
  4. $ docker --help -> app does not appear in the manage commands
  5. $ DOCKER_CLI_EXPERIMENTAL=enabled docker --help -> app appears in the manage commands

- Description for the changelog

  • app plugin is now experimental and only shown if the docker/cli has its experimental flag enabled.

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

image

@silvin-lubecki silvin-lubecki changed the title Mark docker app plugin as experimental [WIP] Mark docker app plugin as experimental May 22, 2019
@silvin-lubecki
Copy link
Contributor Author

@simonferquel can you review the docker desktop context store part, not really sure that what I did is right 🤔

Copy link
Contributor

@ijc ijc left a comment

Choose a reason for hiding this comment

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

LGTM. You edits to the context stuff appear to be consistent with docker/cli@a3efd5d

@silvin-lubecki silvin-lubecki force-pushed the experimental-plugin branch 2 times, most recently from 869f73f to fefc7ae Compare May 24, 2019 14:12
@silvin-lubecki silvin-lubecki changed the title [WIP] Mark docker app plugin as experimental Mark docker app plugin as experimental May 24, 2019
…mental flag enabled will list docker app as a plugin.

Bump docker/cli to the last 19.03

Signed-off-by: Silvin Lubecki <silvin.lubecki@docker.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 May 24, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #537   +/-   ##
=======================================
  Coverage   69.94%   69.94%           
=======================================
  Files          55       55           
  Lines        3101     3101           
=======================================
  Hits         2169     2169           
  Misses        639      639           
  Partials      293      293
Impacted Files Coverage Δ
cmd/docker-app/main.go 63.63% <ø> (ø) ⬆️

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 f5f3873...cbdc839. Read the comment docs.

@silvin-lubecki silvin-lubecki merged commit 73d8674 into docker:master May 24, 2019
@silvin-lubecki silvin-lubecki deleted the experimental-plugin branch May 24, 2019 16:06
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.

5 participants