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

give more time to delete and ask user once #632

Closed
wants to merge 3 commits into from
Closed

Conversation

sjo00
Copy link

@sjo00 sjo00 commented Oct 21, 2019

deleting the old config files from docker.io (18.09.7-0ubuntu1~18.04.4) with aptitude
nukes all the docker files from an existing and running docker-ce installation

Thanks!

deleting the old config files from docker.io (18.09.7-0ubuntu1~18.04.4) with aptitude
nukes all the docker files from existing and running docker-ce installation
@GordonTheTurtle
Copy link
Contributor

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "master" git@github.com:sjo00/docker-ce.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

sjo00 and others added 2 commits October 23, 2019 09:40
deleting the old config files from docker.io (18.09.7-0ubuntu1~18.04.4) with aptitude
nukes all the docker files from existing and running docker-ce installation

Signed-off-by: sjonetat <sjonetat@toppoint.de>
@andrewhsu
Copy link
Contributor

This docker/docker-ce repo is a composite of upstream repos. The upstream repo for this PR's patch is https://github.com/moby/moby. Open your PR there if you still want to propose this change.

@andrewhsu andrewhsu closed this Nov 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
3 participants