-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Conversation
Thanks for referencing the issue @nebojsa94, another small step towards removing those old dependencies! |
@nebojsa94 Thanks for the PR! I don't think we want this pulled in directly from GitHub. This week I'll go ahead and merge a couple of outstanding PRs and publish it. Then we can bump it normally. Sound ok to you? |
@eggplantzzz Of course, hence the draft status. As I mentioned in the description, I'll update the |
@nebojsa94 @iainjreid Just merged and published |
07a830d
to
dc0ed58
Compare
dc0ed58
to
b0b98e5
Compare
Unfortunately it looks like there is some issue in CI. We'll need to take a closer look before merging this. |
Oh whoops, I just realized that this is targeting branch |
Sure, can we first merge #3156 |
I made a comment over on that PR! |
@eggplantzzz Thanks, closing this one. |
The current version of ethpm-registry dependency has unused native modules.
I'll update the version in
package.json
once trufflesuite/ethpm-registry#9 is merged and released