Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

[stable] Backports #8623

Merged
merged 3 commits into from
May 14, 2018
Merged

[stable] Backports #8623

merged 3 commits into from
May 14, 2018

Conversation

andresilva
Copy link
Contributor

@andresilva andresilva commented May 14, 2018 •
edited by 5chdn
Loading

@sorpaas @andresilva
Fix account list double 0x display (#8596)
55ba2d0 
* Remove unused self import

* Fix account list double 0x display
@sorpaas @andresilva
Trace precompiled contracts when the transfer value is not zero (#8486)
4255d4c 
* Trace precompiled contracts when the transfer value is not zero

* Add tests for precompiled CALL tracing

* Use byzantium test machine for the new test

* Add notes in comments on why we don't trace all precompileds

* Use is_transferred instead of transferred
@andresilva andresilva added the A8-backport 🕸 Pull request is already reviewed well in another branch. label May 14, 2018
@andresilva andresilva changed the base branch from master to stable May 14, 2018 11:07
@andresilva andresilva added the M4-core ⛓ Core client code / Rust. label May 14, 2018
@5chdn
Copy link
Contributor

5chdn commented May 14, 2018 •
edited
Loading

Thanks, can you also apply this: #8573 (just CI)

Edit, I just did it.

@5chdn 5chdn mentioned this pull request May 14, 2018
Closed
57 tasks
@5chdn
Gitlab test script fixes (#8573)
5fedce1 
* Exclude /docs from modified files.

* Ensure all references in the working tree are available

* Remove duplicated line from test script
@5chdn 5chdn merged commit 39b9f1e into stable May 14, 2018
@5chdn 5chdn deleted the andre-stable-backports branch May 14, 2018 13:52
@5chdn 5chdn added this to the Patch milestone May 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@5chdn 5chdn 5chdn approved these changes

Assignees
No one assigned
Labels
A8-backport 🕸 Pull request is already reviewed well in another branch. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
Patch
Development

Successfully merging this pull request may close these issues.
3 participants
@andresilva @5chdn @sorpaas