Skip to content

Commit d1e21cf

Browse files
committed
Fix support for 32 bit platforms
Previously running ReTest.jl on 32 bit machines would fail because the `id` variable would have type Int32 (because it's initialized with a literal) but the function it was getting passed to (`resolve!()`) expected a Int64. Also added a 32 bit test to the CI matrix.
1 parent 351d0d8 commit d1e21cf

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/CI.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ jobs:
2424
- { os: ubuntu-latest, version: '1.6', arch: x64}
2525
- { os: ubuntu-latest, version: '^1.7.0-0', arch: x64}
2626
- { os: ubuntu-latest, version: 'nightly', arch: x64}
27+
- { os: ubuntu-latest, version: '1', arch: x86 }
2728
- { os: windows-latest, version: '1', arch: x64}
2829
- { os: macOS-latest, version: '1', arch: x64}
2930

src/ReTest.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1485,7 +1485,7 @@ function fetchtests((mod, pat), verbose, module_header, maxidw; static, strict,
14851485
descwidth = 0
14861486
hasbroken = false
14871487

1488-
id = 1
1488+
id = Int64(1)
14891489
warned = Ref(false)
14901490

14911491
for ts in tests

0 commit comments

Comments
 (0)