Skip to content

fix: wrap MaxInt64 in int64 to support 386 architecture - #20

Merged
mrsimonemms merged 1 commit into
mainfrom
sje/386-support
Oct 18, 2026
Merged

mrsimonemms merged 1 commit into
mainfrom
sje/386-support

Conversation

@mrsimonemms

Copy link
Copy Markdown
Contributor

Description

Related issue

Fixes #

How to test

Checklist

  • This PR links to an issue using Fixes #..., Closes #... or Relates to #...
  • All tests pass
  • Tests have been added or updated where behaviour changed
  • Documentation has been updated where needed

Signed-off-by: Simon Emms <simon@simonemms.com>
@github-actions

Copy link
Copy Markdown

Codex PR Review

No blocking issues. Ship it.

The explicit int64 conversion fixes the 386 overflow without changing behaviour. The CI matrix adds 386 coverage, and existing tests cover the affected validation.

Tests could not run locally because the read-only environment prevented Go from creating its module cache.

@mrsimonemms
mrsimonemms marked this pull request as ready for review October 18, 2026 13:44
@mrsimonemms
mrsimonemms merged commit a3e41d4 into main Oct 18, 2026
9 checks passed
@mrsimonemms
mrsimonemms deleted the sje/386-support branch October 18, 2026 13:45
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.

1 participant