Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Add bootstrapping using a previous version of the CLI #1294

Merged
merged 8 commits into from
Nov 28, 2019

Conversation

frangio
Copy link
Contributor

@frangio frangio commented Nov 27, 2019

As explained in #1293, we're currently using truffle to compile the contracts in this repo, and we want to migrate to our own CLI. There is an issue with using the CLI in the repo itself because of a circular dependency: the compiler depends on lib, and lib depends on the compiler to build itself.

To work around this, we have created a new bootstrap self-contained package that installs a previous version of the CLI. Note that this is not part of the root Yarn workspaces because there were issues with hoisting that resulted in errors.

This PR replaces #1293 because we thought that requiring a globally installed version of the CLI made the development setup cumbersome by not being self-contained, and this would deter external contributions.

Copy link
Contributor

@nventuro nventuro left a comment

Choose a reason for hiding this comment

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

🙌 Amazing! Thanks!

Can you open an issue explaining the issue with the tests that require the truffle.js file to be in the directory?

@nventuro nventuro merged commit 855c2ad into OpenZeppelin:master Nov 28, 2019
@frangio frangio deleted the add-bootstrapping branch November 28, 2019 18:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants