Skip to content
This repository has been archived by the owner on Mar 3, 2021. It is now read-only.

Refactor remix lib - remix-debug, remix-analyzer #1480

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

yann300
Copy link
Collaborator

@yann300 yann300 commented May 7, 2020

  • remove remix-lib dep from remix-solidity
  • remove remix-debugger
  • move classes from remix-lib to remix-debug
  • remove remix-lib dep from remix-analyzer

Most of the changes of this PR are about moving code from remix-lib to remix-debug (TraceManager, CodeManager, ...).
Some helpers and utils that are only used for unit testing are duplicated among projects in order to remove dependency over remix-lib.
That said, some module like Web3VmProvider are still used across different projects (remix-debug and remix-simulator) - this could be addressed in a different PR.

@yann300 yann300 added the WIP Work In Progress label May 7, 2020
@yann300 yann300 force-pushed the refactor_remix_lib2 branch 2 times, most recently from 3af34b2 to c2a9b1b Compare May 7, 2020 13:17
@yann300 yann300 force-pushed the refactor_remix_lib2 branch from c2a9b1b to 8834639 Compare May 7, 2020 13:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
WIP Work In Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant