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 ethers to 5.4.6 / set-protocol-v2@0.1.0 #76

Merged
merged 2 commits into from
Sep 15, 2021

Conversation

cgewecke
Copy link
Contributor

@cgewecke cgewecke commented Sep 8, 2021

Pulls in changes at set-protocol-v2 PR 131 which upgrades ethers to 5.4.6 and uses a new typechain package

Upgrades

  • ethers -> 5.4.6
  • ganache-cli -> 6.12.2
  • typescript -> 4.4.2
  • set-protocol-v2 -> 0.1.0

TODO

Note: There are a handful of breaking TS interface changes here (coming from upstream at set-protocol-v2). When we publish we should bump the version to 0.3.0

@rootulp
Copy link
Contributor

rootulp commented Sep 10, 2021

[ ] attribute commits to rootulp

This is incredibly generous. In full transparency, I contributed a very small amount to SetProtocol/set-protocol-v2#131 but I really appreciate all your hard work to untangle this web of upgrades @cgewecke!

Comment on lines +219 to +225
): Promise<
[BigNumber, BigNumber, BigNumber] & {
totalQuantity: BigNumber;
managerFee: BigNumber;
protocolFee: BigNumber;
}
> {
Copy link
Contributor

Choose a reason for hiding this comment

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

nice tuple type! Since this is public API, I'm sure consumers will appreciate this more precise type definition

@rootulp
Copy link
Contributor

rootulp commented Sep 15, 2021

we should bump the version to 0.3.0

Minor but the last release I see on GH is 0.1.0: https://github.com/SetProtocol/set.js/releases
Even though 0.2.6 appears to exist per package.json
https://github.com/SetProtocol/set.js/blob/master/package.json#L3

So out of curiosity, was the 0.2.x release line blacklisted? Or is 0.2.6 safe to consume and it just wasn't published to the GH releases page?

@cgewecke cgewecke changed the title [WIP] Upgrade ethers to 5.4.6 / set-protocol-v2@london Upgrade ethers to 5.4.6 / set-protocol-v2@0.1.0 Sep 15, 2021
@cgewecke
Copy link
Contributor Author

cgewecke commented Sep 15, 2021

@rootulp The missing release for 0.2.x is an oversight. That version added some trade quote utilities that are used in TokenSets but didn't change anything else about the API here....

Will add a release for that with some usage notes...

Copy link
Contributor

@asoong asoong left a comment

Choose a reason for hiding this comment

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

LGTM

@cgewecke cgewecke merged commit 181ccc6 into master Sep 15, 2021
@cgewecke cgewecke deleted the chris/update-ethers branch September 15, 2021 18:51
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.

3 participants