Skip to content

vendor: github.com/stretchr/testify v1.11.0 - #3378

Merged
crazy-max merged 1 commit into
docker:masterfrom
crazy-max:update-testify
Aug 26, 2025
Merged

crazy-max merged 1 commit into
docker:masterfrom
crazy-max:update-testify

Conversation

@crazy-max

@crazy-max crazy-max commented Aug 25, 2025 •

Copy link
Copy Markdown
Member

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max

Copy link
Copy Markdown
Member Author

Alpine repo is really flaky today:

#42 0.053 fetch https://dl-cdn.alpinelinux.org/alpine/v3.22/main/x86_64/APKINDEX.tar.gz
#42 0.090 fetch https://dl-cdn.alpinelinux.org/alpine/v3.22/community/x86_64/APKINDEX.tar.gz
#42 0.090 WARNING: fetching https://dl-cdn.alpinelinux.org/alpine/v3.22/main: Permission denied
#42 0.392 ERROR: unable to select packages:
#42 0.392   file (no such package):
#42 0.392     required by: world[file]
#42 0.392   git (no such package):
#42 0.392     required by: world[git]

@crazy-max crazy-max added this to the v0.28.0 milestone Aug 25, 2025
@crazy-max
crazy-max marked this pull request as ready for review August 25, 2025 13:53
Comment on lines 1 to -2
//go:build testify_yaml_custom && !testify_yaml_fail && !testify_yaml_default
// +build testify_yaml_custom,!testify_yaml_fail,!testify_yaml_default

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Interesting, but guess it doesn't do much for the go.mod dependencies, as that would still see it. ( see the comment below about "not linking with ..")

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@crazy-max
crazy-max merged commit d7cd677 into docker:master Aug 26, 2025
201 of 203 checks passed
@crazy-max
crazy-max deleted the update-testify branch August 26, 2025 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants