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

ci: test on windows #4293

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

ci: test on windows #4293

wants to merge 12 commits into from

Conversation

caarlos0
Copy link
Member

adding windows tests so I can better investigate some issues

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 14, 2023
@cloudflare-pages
Copy link

cloudflare-pages bot commented Sep 14, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 802fb10
Status: ✅  Deploy successful!
Preview URL: https://0157eae7.goreleaser.pages.dev
Branch Preview URL: https://windows-test-2.goreleaser.pages.dev

View logs

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #4293 (99f4e97) into main (027ddd3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 99f4e97 differs from pull request most recent head 802fb10. Consider uploading reports for the commit 802fb10 to get more accurate results

@@           Coverage Diff           @@
##             main    #4293   +/-   ##
=======================================
  Coverage   84.11%   84.11%           
=======================================
  Files         132      132           
  Lines       12537    12543    +6     
=======================================
+ Hits        10545    10551    +6     
  Misses       1587     1587           
  Partials      405      405           
Files Changed Coverage Δ
internal/client/github.go 59.68% <ø> (ø)
internal/pipe/scoop/scoop.go 75.36% <100.00%> (ø)
internal/pipe/winget/winget.go 83.85% <100.00%> (+0.30%) ⬆️

@pull-request-size pull-request-size bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 14, 2023
@pull-request-size pull-request-size bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 15, 2023
@lincolnthalles
Copy link

Just in case you didn't noticed yet, there's a long-standing issue with the Go testing package on Windows.

Whenever a test uses t.TempDir(), the cleanup fails, and so does the test.

This still affects the latest go 1.21.3.

golang/go#51442
golang/go#52986
golang/go#44919

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants