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

[Go 1.9] deprecate 1.9 & 1.10 versions and introduce 1.12 #921

Merged
merged 1 commit into from
Jun 18, 2019

Conversation

benjaminch
Copy link
Contributor

This CL deprecates Go 1.9 version removing it from the CI along updating
the installation instructions.

This PR probably needs to be flagged as important change so people using Go 1.9 will be aware of this change from now on.

Issue: #920

hhhjort
hhhjort previously approved these changes Jun 6, 2019
Copy link
Contributor

@guscarreon guscarreon left a comment

Choose a reason for hiding this comment

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

Since Go only supports the last two major releases, versions 1.11 and 1.12 are the only ones currently being supported. Do you think we should try to deprecate 1.10.x too?

@benjaminch
Copy link
Contributor Author

Since Go only supports the last two major releases, versions 1.11 and 1.12 are the only ones currently being supported. Do you think we should try to deprecate 1.10.x too?

Sure ! :)

@benjaminch benjaminch changed the title [Go 1.9] deprecate 1.9 version [Go 1.9] deprecate 1.9 & 1.10 versions and introduce 1.12 Jun 10, 2019
This CL deprecates Go 1.9 & 1.10 versions removing it from the CI along updating
the installation instructions and add Go 1.12 support.

Issue: prebid#920
@benjaminch
Copy link
Contributor Author

Hey @guscarreon seems that 1.12 requires a bit more work especially around validate.sh for go vet. I updated it let me know if it works for you.

Cheers!

@guscarreon
Copy link
Contributor

Hey @guscarreon seems that 1.12 requires a bit more work especially around validate.sh for go vet. I updated it let me know if it works for you.

Cheers!

Worked perfeclty. LGTM

Copy link
Contributor

@guscarreon guscarreon left a comment

Choose a reason for hiding this comment

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

Tested soloution with 1.11.1, 1.11.5 and 1.12.6 and no issues were found

@benjaminch
Copy link
Contributor Author

Thanks @guscarreon :)

@mansinahar mansinahar merged commit b1829f0 into prebid:master Jun 18, 2019
mansinahar added a commit that referenced this pull request Jun 18, 2019
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 1, 2020
This CL deprecates Go 1.9 & 1.10 versions removing it from the CI along updating
the installation instructions and add Go 1.12 support.

Issue: prebid#920
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 2, 2020
This CL deprecates Go 1.9 & 1.10 versions removing it from the CI along updating
the installation instructions and add Go 1.12 support.

Issue: prebid#920
katsuo5 pushed a commit to flux-dev-team/prebid-server-1 that referenced this pull request Dec 4, 2020
This CL deprecates Go 1.9 & 1.10 versions removing it from the CI along updating
the installation instructions and add Go 1.12 support.

Issue: prebid#920
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