Skip to content
This repository was archived by the owner on Jul 31, 2025. It is now read-only.

Conversation

@justincormack
Copy link
Contributor

This now uses the standard Go context package rather than
gorilla-context expect on Go 1.6 and earlier. As the context
packages should not be mixed this is better.

Have not removed gorilla-context as vndr still thinks it is
required, nut it will not be used.

Signed-off-by: Justin Cormack [email protected]

This now uses the standard Go context package rather than
gorilla-context expect on Go 1.6 and earlier. As the context
packages should not be mixed this is better.

Have not removed gorilla-context as `vndr` still thinks it is
required, nut it will not be used.

Signed-off-by: Justin Cormack <[email protected]>
Copy link
Contributor

@riyazdf riyazdf left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ecordell ecordell left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@cyli cyli left a comment

Choose a reason for hiding this comment

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

LGTM

@cyli cyli merged commit 39e32b3 into notaryproject:master Apr 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants