Skip to content

ci: remove 386 (32 bit x86)#5352

Merged
marten-seemann merged 1 commit intoquic-go:masterfrom
MarcoPolo:push-pvpxqqqkvlol
Oct 6, 2025
Merged

ci: remove 386 (32 bit x86)#5352
marten-seemann merged 1 commit intoquic-go:masterfrom
MarcoPolo:push-pvpxqqqkvlol

Conversation

@MarcoPolo
Copy link
Copy Markdown
Collaborator

The last 386 cpu was from 20 years ago (Pentium 4)

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.33%. Comparing base (97f3aae) to head (876057b).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5352      +/-   ##
==========================================
+ Coverage   84.23%   84.33%   +0.10%     
==========================================
  Files         156      157       +1     
  Lines       17904    18008     +104     
==========================================
+ Hits        15081    15187     +106     
+ Misses       2248     2245       -3     
- Partials      575      576       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@marten-seemann marten-seemann left a comment

Choose a reason for hiding this comment

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

What about the unit tests (unit.yml). There's a 32 bit step there as well.

Comment thread .github/workflows/cross-compile.yml Outdated
Comment thread .github/workflows/integration.yml
@marten-seemann
Copy link
Copy Markdown
Member

@MarcoPolo Are you planning to address the remaining review comments?

The last 386 cpu was from 20 years ago (Pentium 4)
@MarcoPolo
Copy link
Copy Markdown
Collaborator Author

ah sorry, I thought I had pushed

Copy link
Copy Markdown
Member

@marten-seemann marten-seemann left a comment

Choose a reason for hiding this comment

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

Thank you!

@marten-seemann marten-seemann changed the title ci: remove 386 (32bit x86) from CI ci: remove 32 bit tests from CI Oct 6, 2025
@marten-seemann marten-seemann changed the title ci: remove 32 bit tests from CI ci: remove 32 bit tests Oct 6, 2025
@marten-seemann marten-seemann changed the title ci: remove 32 bit tests ci: remove 386 (32 bit x86) Oct 6, 2025
@marten-seemann marten-seemann merged commit 73e8240 into quic-go:master Oct 6, 2025
19 checks passed
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.

2 participants