Skip to content

Conversation

@stevvooe
Copy link
Contributor

@stevvooe stevvooe commented Jan 9, 2017

Aaron and Derek have long been instrumental in the maintenance and
integration of the original digest package. Since this package
provides such critical functionality, having their experience on board
will ensure that additions to this package can be made with confidence.

Signed-off-by: Stephen J Day [email protected]

Aaron and Derek have long been instrumental in the maintenance and
integration of the original `digest` package. Since this package
provides such critical functionality, having their experience on board
will ensure that additions to this package can be made with confidence.

Signed-off-by: Stephen J Day <[email protected]>
@caniszczyk
Copy link
Contributor

lets procure other LGTMs from the @opencontainers/go-digest-maintainers

@vbatts
Copy link
Member

vbatts commented Jan 10, 2017

LGTM
I've worked with aaron and derek for a while, and look forward to again.

Approved with PullApprove

@jonboulle
Copy link
Contributor

jonboulle commented Jan 10, 2017

lgtm

Approved with PullApprove

@jonboulle jonboulle merged commit 7ecb13d into opencontainers:master Jan 10, 2017
@wking
Copy link
Contributor

wking commented Jan 10, 2017 via email

@stevvooe
Copy link
Contributor Author

stevvooe commented Jan 10, 2017

LGTM

Approved with PullApprove

@stevvooe
Copy link
Contributor Author

@opencontainers/go-digest-maintainers Please mark this with your LGTM. If this doesn't pass, we'll merge #26 and revert the change.

@brendandburns
Copy link

brendandburns commented Jan 12, 2017

LGTM.

Approved with PullApprove

@jstarks
Copy link

jstarks commented Jan 13, 2017

LGTM

Approved with PullApprove

wking added a commit to wking/nmbug-oci that referenced this pull request Feb 1, 2017
Most of the voting happened in the GitHub PR [1], which passed +5 -0
#2, with Brandon Philips and Jason Bouzane not weighing in.

[1]: opencontainers/go-digest#25
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.

7 participants