Skip to content

Conversation

@andrewhsu
Copy link
Contributor

@andrewhsu andrewhsu commented Jun 18, 2019

added deprecation notice for registry v2 schema 1

related: moby/moby#39365

@codecov-io
Copy link

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1956   +/-   ##
=======================================
  Coverage   56.72%   56.72%           
=======================================
  Files         310      310           
  Lines       21800    21800           
=======================================
  Hits        12367    12367           
  Misses       8518     8518           
  Partials      915      915

@codecov-io
Copy link

codecov-io commented Jun 18, 2019

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1956   +/-   ##
=======================================
  Coverage   56.72%   56.72%           
=======================================
  Files         310      310           
  Lines       21800    21800           
=======================================
  Hits        12367    12367           
  Misses       8518     8518           
  Partials      915      915

@andrewhsu andrewhsu marked this pull request as ready for review June 19, 2019 03:14
@andrewhsu andrewhsu requested a review from thaJeztah as a code owner June 19, 2019 03:14
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Member

prepared backport; #1960

@GordonTheTurtle
Copy link

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 "deprecate" [email protected]:andrewhsu/cli.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842356989440
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.

@tiborvass
Copy link
Collaborator

@andrewhsu I can't modify your branch, did you keep the "allow edits from maintainers" checkbox when you opened the PR ?

Co-Authored-By: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Andrew Hsu <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member

squashed the commits

@thaJeztah thaJeztah merged commit 44578e5 into docker:master Jun 24, 2019
@GordonTheTurtle GordonTheTurtle added this to the 19.09.0 milestone Jun 24, 2019
@andrewhsu andrewhsu deleted the deprecate branch October 11, 2019 21:00
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.

7 participants