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

Refactor Plan #395

Open
pipermerriam opened this Issue Dec 1, 2017 · 0 comments

Comments

Projects
None yet
1 participant
@pipermerriam
Copy link
Member

pipermerriam commented Dec 1, 2017

This is an attempt to document the plan for pending refactors to Populus.

  • Change populus to use eth-tester for testing.
  • Remove need for project.json in project directory.
  • Don't default to populating ~/.populus/config.json with the default populus config.
  • Remove $refs from config
  • Change to have an empty project.json result in the same current config.
  • Support for multiple compilers in the same project.
  • Remove the Chain API entirely.
  • Remove the scripts that setup local test chains.
  • Remove py-geth as a dependency.
  • Move web3 pytest fixtures out of Populus and into eth-tester or maybe web3.py
  • Change compiled asset format to be valid minimal ERC190 package.
  • Refactor Contract Backend API (TODO: flesh this out)

Things to figure out:

  • How can the populus config be flattened?
  • How does config deal with the situation where there are multiple web3 connections to the same chain and they all need to share configuration values like the address of the ENS resolver or the address of a package registry.
  • What parts of the linking logic can be moved down into the Contract object.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment