Skip to content

Make dockerpycreds part of the SDK under docker.credentials#2222

Merged
shin- merged 2 commits intomasterfrom
vendor_dockerpycreds
May 1, 2019
Merged

Make dockerpycreds part of the SDK under docker.credentials#2222
shin- merged 2 commits intomasterfrom
vendor_dockerpycreds

Conversation

@shin-
Copy link
Copy Markdown
Contributor

@shin- shin- commented Jan 10, 2019

No description provided.

@shin- shin- added this to the 4.0.0 milestone Jan 10, 2019
@GordonTheTurtle
Copy link
Copy Markdown

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 "vendor_dockerpycreds" git@github.com:docker/docker-py.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842359069392
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

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

@shin- shin- force-pushed the vendor_dockerpycreds branch from 9ccd3f4 to e3787bb Compare April 14, 2019 22:18
Copy link
Copy Markdown
Member

@chris-crone chris-crone left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @shin-. We should mark on the docker py creds repo as no longer maintained and point people here.

shin- added 2 commits April 30, 2019 23:37
Signed-off-by: Joffrey F <joffrey@docker.com>
Signed-off-by: Joffrey F <joffrey@docker.com>
@shin- shin- force-pushed the vendor_dockerpycreds branch from e3787bb to 4c45067 Compare May 1, 2019 06:38
@shin- shin- merged commit cc13b82 into master May 1, 2019
@shin- shin- deleted the vendor_dockerpycreds branch May 1, 2019 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants