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.

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

Update master with changes from next #553

Merged
merged 4 commits into from Sep 3, 2019

Conversation

@facuspagnuolo
Copy link
Member

commented Sep 3, 2019

No description provided.

sohkai and others added 4 commits Jul 15, 2019
DepositableDelegateProxy: optimize for EIP-1884 (#551)
* DepositableDelegateProxy: optimize for EIP-1884

* chore: pin solidity-coverage to v0.6.2 (#552)

Co-Authored-By: Brett Sun <qisheng.brett.sun@gmail.com>

@facuspagnuolo facuspagnuolo requested a review from izqui Sep 3, 2019

@auto-assign auto-assign bot requested a review from sohkai Sep 3, 2019

@izqui
izqui approved these changes Sep 3, 2019
@sohkai
sohkai approved these changes Sep 3, 2019
Copy link
Member

left a comment

LGTM!

Good thing the SafeERC20 changes are / should be backwards compatible :D

@coveralls

This comment has been minimized.

Copy link

commented Sep 3, 2019

Coverage Status

Coverage decreased (-0.4%) to 99.116% when pulling 0df3c81 on next into c85d34e on master.

@facuspagnuolo facuspagnuolo merged commit 2bff3be into master Sep 3, 2019

5 of 7 checks passed

coverage/coveralls Coverage decreased (-0.4%) to 99.116%
Details
License Compliance FOSSA is analyzing this commit
Details
Gas Usage New gas usage report!
Details
WIP Ready for review
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
license/cla Contributor License Agreement is signed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.