search

Glow-Lang / glow

The Glow language for Blockchain Decentralized Applications.
https://glow-lang.org
Apache License 2.0
58 stars 8 forks source link

Track contract balance manually in global var. - [merged] #350

Closed kwannoel closed 3 years ago

kwannoel commented 3 years ago

In GitLab by @isd on May 25, 2021, 07:01

Merges issue-185 -> master

Related gerbil-ethereum patch:

https://github.com/fare/gerbil-ethereum/pull/38

...which will have to be merged before this patch will work, but the integration tests pass on my local dev setup.

This patch does the glow side of that -- manages restoring the merkelized balance variable.

Fixes #185

Note, this is already rebased on top of !139, which still needs to be merged.

kwannoel commented 3 years ago

In GitLab by @isd on May 25, 2021, 07:01

requested review from @fahree

kwannoel commented 3 years ago

In GitLab by @fahree on May 27, 2021, 07:22

You may have to update the dependencies now that the gerbil-ethereum PR was merged.

kwannoel commented 3 years ago

In GitLab by @fahree on May 27, 2021, 07:23

approved this merge request

kwannoel commented 3 years ago

In GitLab by @isd on May 29, 2021, 01:36

added 14 commits

Compare with previous version

kwannoel commented 3 years ago

In GitLab by @isd on May 29, 2021, 01:39

enabled an automatic merge when the pipeline for 55c311e64e976a6ad4abcca7abdedb9e03ea7d7e succeeds

kwannoel commented 3 years ago

In GitLab by @isd on May 29, 2021, 01:40

Updated gerbil-ethreum dependency, rebased, and set this to merge automatically once CI succeeds.

kwannoel commented 3 years ago

In GitLab by @isd on May 29, 2021, 01:54

mentioned in commit bceab5553812eb8598f84c922d8ea888b038cff3