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

build: test on windows too #3795

Closed
wants to merge 17 commits into from
Closed

build: test on windows too #3795

wants to merge 17 commits into from

Conversation

caarlos0
Copy link
Member

this ought to be painful, but its time, probably...

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 22, 2023
@caarlos0 caarlos0 self-assigned this Feb 22, 2023
@caarlos0 caarlos0 added enhancement New feature or request and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 22, 2023
@cloudflare-pages
Copy link

cloudflare-pages bot commented Feb 22, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9528f0c
Status: ✅  Deploy successful!
Preview URL: https://19cb888e.goreleaser.pages.dev
Branch Preview URL: https://windows.goreleaser.pages.dev

View logs

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 22, 2023
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Feb 22, 2023

Codecov Report

Merging #3795 (40f9d99) into main (94981fb) will decrease coverage by 0.10%.
The diff coverage is 66.66%.

❗ Current head 40f9d99 differs from pull request most recent head 9528f0c. Consider uploading reports for the commit 9528f0c to get more accurate results

@@            Coverage Diff             @@
##             main    #3795      +/-   ##
==========================================
- Coverage   83.64%   83.54%   -0.10%     
==========================================
  Files         130      124       -6     
  Lines       12216    10960    -1256     
==========================================
- Hits        10218     9157    -1061     
+ Misses       1597     1445     -152     
+ Partials      401      358      -43     
Impacted Files Coverage Δ
internal/testlib/path.go 78.57% <40.00%> (-21.43%) ⬇️
internal/archivefiles/archivefiles.go 92.55% <100.00%> (+1.89%) ⬆️
internal/artifact/artifact.go 84.96% <100.00%> (+0.65%) ⬆️
internal/pipe/scoop/scoop.go 68.89% <100.00%> (-7.74%) ⬇️

... and 54 files with indirect coverage changes

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@pull-request-size pull-request-size bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Feb 22, 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/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Feb 22, 2023
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
caarlos0 added a commit that referenced this pull request Jun 16, 2023
improved test check

extracted from #3795
caarlos0 added a commit that referenced this pull request Jun 16, 2023
this is needed when we run part of the pipeline on windows, and the rest
on linux, for example, the split build pro feature.

extracted from #3795 

+ added tests for relative path thing

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
caarlos0 added a commit that referenced this pull request Jun 16, 2023
extracted from #3795

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
caarlos0 added a commit that referenced this pull request Jun 16, 2023
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
caarlos0 added a commit that referenced this pull request Jun 16, 2023
extracted from #3795

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
caarlos0 added a commit that referenced this pull request Jun 17, 2023
@caarlos0
Copy link
Member Author

being continued in #4293

@caarlos0 caarlos0 closed this Sep 20, 2023
@caarlos0 caarlos0 deleted the windows branch September 20, 2023 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 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

1 participant