Skip to content

fix: Compiling superblock.go for 32-bit systems - #55

Merged
raharper merged 1 commit into
project-machine:mainfrom
sertonix:superblock-32bit
Oct 1, 2026
Merged

raharper merged 1 commit into
project-machine:mainfrom
sertonix:superblock-32bit

Conversation

@sertonix

@sertonix sertonix commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

The value is supposed to be uint32 but go attempted to format it as int32 for 32-bit systems.

Ref project-stacker/stacker#648

The value is uint32 but go attempted to format it as int32 for 32-bit
systems.

Signed-off-by: Sertonix <sertonix@posteo.net>
@codecov

codecov Bot commented Oct 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 37.21%. Comparing base (e06accf) to head (17b9620).

Files with missing lines Patch % Lines
pkg/erofs/superblock.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   37.21%   37.21%           
=======================================
  Files          24       24           
  Lines        2257     2257           
=======================================
  Hits          840      840           
  Misses       1248     1248           
  Partials      169      169           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@raharper raharper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@raharper
raharper merged commit 366d300 into project-machine:main Oct 1, 2026
6 of 7 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