Skip to content

build: upgrade go directive in go.mod to 1.17 - #2219

Merged
gmlewis merged 1 commit into
google:masterfrom
Juneezee:go1.17
Dec 6, 2021
Merged

build: upgrade go directive in go.mod to 1.17#2219
gmlewis merged 1 commit into
google:masterfrom
Juneezee:go1.17

Conversation

@Juneezee

Copy link
Copy Markdown
Contributor

This PR upgrades the go directive in go.mod file by running go mod tidy -go=1.17 to enable module graph pruning and lazy module loading.

Note 1: This does not prevent users with earlier Go versions from successfully building packages from this module.

Note 2: The additional require directive is used to record indirect dependencies for Go 1.17 or higher, see https://go.dev/ref/mod#go-mod-file-go.

This commit enables support for module graph pruning and lazy module
loading for projects that are at Go 1.17 or higher.
Reference: https://go.dev/ref/mod#go-mod-file-go
Reference: https://go.dev/ref/mod#graph-pruning
Reference: https://go.dev/ref/mod#lazy-loading
Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>
@google-clagoogle-claBot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Dec 6, 2021
@codecov

codecovBot commented Dec 6, 2021

Copy link
Copy Markdown

Codecov Report

Merging #2219 (ef5fd04) into master (f5943db) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## master #2219 +/- ##
=======================================
Coverage 97.81% 97.81% =======================================
Files 112 112 Lines 10117 10117 =======================================
Hits 9896 9896 Misses 154 154 Partials 67 67 

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5943db...ef5fd04. Read the comment docs.

@gmlewisgmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @Juneezee !
LGTM.
Merging.

@gmlewis
gmlewis merged commit 7520506 into google:masterDec 6, 2021
jlaportebot added a commit to jlaportebot/go-github that referenced this pull request Jun 28, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesIndication that the PR author has signed a Google Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Juneezee@gmlewis