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

Upgrade eth-tester to v0.4.0-beta.2 #1636

Merged
merged 2 commits into from
May 7, 2020

Conversation

kclowes
Copy link
Collaborator

@kclowes kclowes commented May 1, 2020

What was wrong?

With pip's lack of dependency resolution, we had an incompatible eth-keys version that was raising errors in a fresh pip install -e .[dev] install, so it had to be pinned at "eth-keys>=0.2.1,<0.3". This is fixed in the v0.4.0-beta2 release of eth-tester.

Related to issue #1622

How was it fixed?

Upgraded eth-tester

Todo:

Cute Animal Picture

image

@kclowes kclowes force-pushed the eth-tester-040b2 branch from 3da5917 to b170e58 Compare May 1, 2020 17:09
@kclowes kclowes changed the title [WIP] Upgrade eth-tester to v0.4.0-beta.2 Upgrade eth-tester to v0.4.0-beta.2 May 1, 2020
@kclowes kclowes requested review from carver and wolovim May 1, 2020 17:10
docs/contracts.rst Outdated Show resolved Hide resolved
@kclowes kclowes force-pushed the eth-tester-040b2 branch from 0fcf80e to 3b28161 Compare May 6, 2020 21:12
Copy link
Collaborator

@carver carver left a comment

Choose a reason for hiding this comment

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

Yup, looks good to me! A couple little quirks in the array_contract doctest, but nothing that I should re-review, unless you want me to. 👍

docs/contracts.rst Outdated Show resolved Hide resolved
docs/contracts.rst Outdated Show resolved Hide resolved
@kclowes kclowes force-pushed the eth-tester-040b2 branch 2 times, most recently from d01c8d1 to e29e7b2 Compare May 7, 2020 21:20
@kclowes kclowes force-pushed the eth-tester-040b2 branch from e29e7b2 to 1a53ff5 Compare May 7, 2020 21:44
@kclowes kclowes force-pushed the eth-tester-040b2 branch from 1a53ff5 to 2d1804d Compare May 7, 2020 21:47
@kclowes kclowes merged commit 4a9cf76 into ethereum:master May 7, 2020
@kclowes kclowes deleted the eth-tester-040b2 branch May 7, 2020 21:54
@kclowes kclowes mentioned this pull request Jun 2, 2020
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants