Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Develop #44

Merged
merged 32 commits into from
Oct 6, 2022
Merged

Develop #44

merged 32 commits into from
Oct 6, 2022

Conversation

krlosMata
Copy link
Contributor

No description provided.

@cla-bot cla-bot bot added the cla-signed label Sep 21, 2022
@krlosMata krlosMata marked this pull request as draft September 21, 2022 11:09
zkronos73 and others added 25 commits September 21, 2022 13:18
@zkronos73
update scripts and versions
4efaa6a
@agnusmor
Remove funcName "isSetReplacingZero" as it is not used in the rom Sto…
0556aaa 
…rage.

Update storage_sm_rom.json (fix typo "GetRkey")
@zkronos73
steps feature to extend execution out n in debug mode
dbbf099
@zkronos73
on debug skip checkFinalState control
79f528f
@zkronos73
fastDebugExit flag
a011b9c
@zkronos73
Merge pull request #46 from 0xPolygonHermez/feature/steps-vs-n
052b73b 
steps feature to extend execution out n in debug mode
@ignasirv
Small acessedStorage fixes
7813ed8 
typo + instantiation
@laisolizq
remove old tracer. Full-tracer actuvated with -T flag. Update run-inputs
e50bf57
@zkronos73
Merge pull request #45 from 0xPolygonHermez/remove-issetreplacingzero
7f46141 
Remove "isSetReplacingZero". Update storage_sm_rom.json
@invocamanman
fix mem tracer
cdb625f
@krlosMata
Merge pull request #48 from 0xPolygonHermez/feature/acessedStorage-fix
d2507d3 
Feature/acessed storage fix
@krlosMata @ignasirv
run-inputs with 22 steps
5f46c26
@krlosMata @ignasirv
remove old tracer. Full-tracer actovated with -t flag
afcaa5e
@ignasirv
Add scripts to run tests in parallel from GHA
e8d1d20
@ignasirv
Set stepsN for out of memory issues
f31be9f
@krlosMata
Merge pull request #52 from 0xPolygonHermez/feature/parallel-tests
94989a7 
Feature/parallel tests
@ignasirv
Apply changes from tracer cpp
3612ca1
@krlosMata
Merge pull request #50 from 0xPolygonHermez/feature/fixMemTracer
5fc4a9d 
fix mem tracer
@krlosMata
Merge pull request #53 from 0xPolygonHermez/feature/tracer-gas-limit
3511691 
Apply changes from tracer cpp
@ignasirv
Add input param chainId
89c499c
@krlosMata
adapt tool build-genesis
aea29b6
@ignasirv
Update commonjs version
a42d960
@zkronos73
Merge pull request #54 from 0xPolygonHermez/feature/chain-id
ee26e82 
Add input param chainId
@zkronos73
improve storage test and tests tools
ba3141c
@zkronos73
set versions and dependencies on package.json, add --bctree option
9ec4e23
eduadiez
eduadiez approved these changes Oct 6, 2022
View reviewed changes
Copy link
Contributor

@eduadiez eduadiez left a comment

Choose a reason for hiding this comment

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

LGTM

@zkronos73 zkronos73 marked this pull request as ready for review October 6, 2022 07:56
@zkronos73 zkronos73 merged commit 9dc19a5 into main Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eduadiez eduadiez eduadiez approved these changes

@zkronos73 zkronos73 zkronos73 approved these changes

@ignasirv ignasirv Awaiting requested review from ignasirv

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@krlosMata @eduadiez @zkronos73 @ignasirv @agnusmor @laisolizq @invocamanman