search

tact-lang / ton-opcode

Decompiler of BOC files
MIT License
18 stars 6 forks source link

add CI testing #8

Closed Gusarich closed 3 months ago

Gusarich commented 4 months ago

it fails for some reason when testing with tact. might https://github.com/tact-lang/ton-opcode/commit/82dcf9cd065084cb84cc2114c5e1bb3e4a6f9f96 be the reason?

💼 Compiling project map-traverse...
   > MapTraverseTestContract: tact compiler
   > MapTraverseTestContract: func compiler
   > MapTraverseTestContract: fift decompiler
Fift decompiler crashed
Error: Offset(0) > 0 is out of bounds
    at BitString.substring (/home/runner/work/ton-opcode/ton-opcode/node_modules/@ton/core/dist/boc/BitString.js:72:19)
    at BitReader.loadBits (/home/runner/work/ton-opcode/ton-opcode/node_modules/@ton/core/dist/boc/BitReader.js:83:28)
    at Slice.loadBits (/home/runner/work/ton-opcode/ton-opcode/node_modules/@ton/core/dist/boc/Slice.js:100:29)
    at subslice (/home/runner/work/ton-opcode/ton-opcode/dist/utils/subcell.js:25:19)
    at decompile (/home/runner/work/ton-opcode/ton-opcode/dist/decompiler/decompiler.js:33:40)
    at decompileCell (/home/runner/work/ton-opcode/ton-opcode/dist/decompiler/decompileAll.js:13:48)
    at /home/runner/work/ton-opcode/ton-opcode/dist/decompiler/decompileAll.js:171:17
    at Writer.inIndent (/home/runner/work/ton-opcode/ton-opcode/dist/utils/Writer.js:13:13)
    at decompileCell (/home/runner/work/ton-opcode/ton-opcode/dist/decompiler/decompileAll.js:170:20)
    at /home/runner/work/ton-opcode/ton-opcode/dist/decompiler/decompileAll.js:44:21
💥 Compilation failed. Skipping packaging

https://github.com/Gusarich/ton-opcode/actions/runs/9020580130/job/24786043758

Gusarich commented 3 months ago

@anton-trunov ready to merge

anton-trunov commented 3 months ago

CI is actually failing: https://github.com/tact-lang/ton-opcode/actions/runs/9410040546

Gusarich commented 3 months ago

CI is actually failing: https://github.com/tact-lang/ton-opcode/actions/runs/9410040546

I see, I'll fix that ASAP

anton-trunov commented 3 months ago

Looks like I already fixed it in #11 :)

anton-trunov commented 3 months ago

But please do double check if that's the right fix