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

update gotestsum to v1.8.2 #3869

Merged
merged 1 commit into from Nov 18, 2022
Merged

Conversation

thaJeztah
Copy link
Member

release notes: https://github.com/gotestyourself/gotestsum/releases/tag/v1.8.2

  • Show shuffle seed
  • Update tests, and cleanup formats
  • Update dependencies
  • Test against go1.19, remove go1.15
  • Add project name to junit.xml output
  • Adding in support for s390x and ppc64le

full diff: gotestyourself/gotestsum@v1.8.1...v1.8.2

- A picture of a cute animal (not mandatory but encouraged)

@codecov-commenter
Copy link

codecov-commenter commented Nov 17, 2022

Codecov Report

Merging #3869 (7000991) into master (da7bf25) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3869   +/-   ##
=======================================
  Coverage   59.22%   59.22%           
=======================================
  Files         288      288           
  Lines       24661    24661           
=======================================
  Hits        14606    14606           
  Misses       9171     9171           
  Partials      884      884           

@thaJeztah
Copy link
Member Author

Oh, fun; probably because newer go version now expect git to be present 😞

 > [gotestsum 1/1] RUN --mount=type=cache,target=/root/.cache/go-build     --mount=type=cache,target=/go/pkg/mod     --mount=type=tmpfs,target=/go/src/     GO111MODULE=on go install gotest.tools/gotestsum@v0.8.2:
#19 2.947 go: gotest.tools/gotestsum@v0.8.2: git init --bare in /go/pkg/mod/cache/vcs/919b7aefa288fd1cde238e8bf4522b7b73294f2de0b84bc742227fffb6f45d60: exec: "git": executable file not found in $PATH

@thaJeztah
Copy link
Member Author

Was about to add -buildvcs=false, but can't reproduce the issue locally; maybe it was just a hiccup

@thaJeztah
Copy link
Member Author

🤦 looking at the wrong Dockerfile; need to look at Dockerfile.dev

release notes: https://github.com/gotestyourself/gotestsum/releases/tag/v1.8.2

- Show shuffle seed
- Update tests, and cleanup formats
- Update dependencies
- Test against go1.19, remove go1.15
- Add project name to junit.xml output
- Adding in support for s390x and ppc64le

full diff: gotestyourself/gotestsum@v1.8.1...v1.8.2

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@@ -17,7 +17,7 @@ RUN --mount=type=cache,target=/root/.cache/go-build \
&& gofumpt --version

FROM golang AS gotestsum
ARG GOTESTSUM_VERSION=v0.4.0
ARG GOTESTSUM_VERSION=v0.8.2
Copy link
Member Author

Choose a reason for hiding this comment

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

😂

Suggested change
ARG GOTESTSUM_VERSION=v0.8.2
ARG GOTESTSUM_VERSION=v1.8.2

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.

None yet

3 participants