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

Update Lerna & other small changes #896

Merged
merged 9 commits into from
Jul 19, 2018
Merged

Update Lerna & other small changes #896

merged 9 commits into from
Jul 19, 2018

Conversation

fabioberger
Copy link
Contributor

Description

This PR:

  • Upgrades Lerna to 3.0-Beta14
  • Removes the lerna-get-packages dep, and replaces it with our own implementation.
  • Rename variables and types to omit Lerna, since they are actually Lerna independent.

Testing instructions

Run all the scripts in monorepo-scripts package.

Types of changes

Dep upgrade & removal.

Checklist:

  • Change requires a change to the documentation.
  • Added tests to cover my changes.
  • Added new entries to the relevant CHANGELOG.jsons.
  • Labeled this PR with the 'WIP' label if it is a work in progress.
  • Labeled this PR with the labels corresponding to the changed package.

@fabioberger fabioberger changed the title Update Lerna Update Lerna & other small changes Jul 19, 2018
* v2-prototype: (48 commits)
  Update CHANGELOG
  Rename call data schema id to CallData. Check for TypedArray when hashing data in order-utils crypto
  Fix broken links in sol-cov documentation
  Fix 0x.js ts warnings
  Update yarn.lock
  Fix 0x.js ts warnings
  Fix 0x.js tests on CI
  Fix a bad merge
  Update package versions
  Merge
  Update changelogs
  Add a test for ERC721 Allowance
  Use allowance instead of approval for all in fill-scenarios
  Upgrade sha3 to 1.2.2 to work with node v10
  Check if the token doesn't exist before minting in fill scenarios
  Make downlevelIteration a global config
  Fix tests descriptions
  DRY up the code in order-watcher collision-resistant abi decoder
  Await transactions in fillScenarios
  Rename decodeAssetData to decodeAssetDataOrThrow
  ...
@coveralls
Copy link

coveralls commented Jul 19, 2018

Coverage Status

Coverage remained the same at 84.106% when pulling 9ab6ab1 on update-lerna into 886a03f on v2-prototype.

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.

3 participants