Closed fvictorio closed 1 year ago
Latest commit: 2f70305fd84b5ebbd3851392850b000608114334
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Can we get this resolved please, it's a really annoying bug that prevents me to upgrade hardhat-ethers
. @veljko-matic @dule-git
Hey, thanks for contributing! Sorry it took us so long! Closing this issue since we resolved it in this PR.
@dule-git thx - can confirm that fixed my issue.
Hi, this repo has two peer dependencies (
hardhat
andtenderly
) that are also plain dependencies. This is usually an error and can be problematic. For example, @pcaversaccio's template repo now gets an error if you try to upgrade an unrelated package (hardhat-ethers
) because of that.This PR fixes the problem by moving Hardhat to
devDependencies
, and by removingtenderly
as a peer dependency (so it's only a plain dependency now). Hardhat as a peer + dev dependency is the common pattern for plugins. As for thetenderly
package, it could also be a peer + dev dependency if you want people to be able to upgrade it independently, but I think for this particular plugin it might not be necessary. I could be wrong here though, let me know.