Closed zoeyTM closed 1 week ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
hardhat | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jun 24, 2024 3:16pm |
Latest commit: ab95f71617f2c76b138038242c681aa8073af7c6
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
Fixes https://github.com/NomicFoundation/hardhat/issues/5410
Ideally, this PR would split the tests into appropriate tests of
parseTask
andparseTaskArguments
, however, there are many tests and the logic for testing both is deeply intertwined in both. Since the underlying logic is the same, I decided to simply makeparseTaskAndArguments
a test-only function abstraction since I don't think it's worth the dev time right now to fully break out all the tests.