Skip to content

go.mod: github.com/Microsoft/go-winio v0.4.17#5350

Merged
dmcgowan merged 1 commit intocontainerd:masterfrom
thaJeztah:tagged_go_winio
Apr 14, 2021
Merged

go.mod: github.com/Microsoft/go-winio v0.4.17#5350
dmcgowan merged 1 commit intocontainerd:masterfrom
thaJeztah:tagged_go_winio

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

relates to #5345

no code changes as this is the same commit, but now tagged:

microsoft/go-winio@5516f17...v0.4.17

no code changes as this is the same commit, but now tagged:

microsoft/go-winio@5516f17...v0.4.17

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

@kevpar @dims @dmcgowan ptal

Copy link
Copy Markdown
Contributor

@Zyqsempai Zyqsempai left a comment

Choose a reason for hiding this comment

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

LGTM

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Apr 13, 2021

Build succeeded.

@kevpar
Copy link
Copy Markdown
Member

kevpar commented Apr 13, 2021

Looks like none of the code actually used by containerd changed, thus no vendor changes?

Copy link
Copy Markdown
Member

@kevpar kevpar left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Copy Markdown
Member Author

Looks like none of the code actually used by containerd changed, thus no vendor changes?

Correct; we were already on the commit that was tagged, so this only updates the go.mod to match the tag instead of the commit / pseudo version.

One motivation for tagged versions is that distro packagers usually will only use tagged versions; which can lead to obscure errors / issues that may not be in "upstream" (this repository). in this case they would likely have downgraded the dependency to v0.4.16

@dmcgowan dmcgowan merged commit 824b610 into containerd:master Apr 14, 2021
@thaJeztah thaJeztah deleted the tagged_go_winio branch April 14, 2021 06:55
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.

5 participants