Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch from x/net/context to context #133

Merged
merged 2 commits into from
Sep 18, 2018
Merged

Conversation

kolyshkin
Copy link
Contributor

Since go 1.7, "context" is a standard package. Since go 1.9,
x/net/context merelyprovides some types aliased to those in the
standard context package, so there is no functional change.

PS is there any "the most trivial PR of the year" contest? I'd like to nominate this one please.

Since go 1.7, "context" is a standard package. Since go 1.9,
x/net/context merelyprovides some types aliased to those in the
standard context package, so there is no functional change.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Go 1.8 is unsupported since Go 1.10 was released in Feb 2018.

In fact, Go 1.9 is also unsupported (since Go 1.11 was released
in 24 August 2018), but let it stay for now.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin
Copy link
Contributor Author

kolyshkin commented Sep 18, 2018

Also remove go 1.8 from travis.yaml. Alas, this is no longer the post trivial PR 😿

Copy link
Member

@dmcgowan dmcgowan 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
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@estesp estesp merged commit f04dbc0 into containerd:master Sep 18, 2018
@kolyshkin kolyshkin deleted the context branch September 19, 2018 00:00
@AkihiroSuda
Copy link
Member

PS is there any "the most trivial PR of the year" contest? I'd like to nominate this one please.

My PR is more trivial (waiting for review ^^) #127

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