Migrate hardhat ignition to hardhat monorepo #6066
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Update the deployment complete displays for error messages as well. Adds some tests to allow us to iterate quickly.
Redisplay batches on every future init and complete event. Not all events were triggering a redisplay of the batch, now both init and complete events do. Init events where overwriting the batch init for futures, and setting them to pending (aka timeout). The fix is to set them to unstarted (the same as the batch init). This fixes the flash of pending/timed out displayed for futures. Fixes #545.
Update the different status displays to match the new styling of deploy. The status display has been separated out from the console logging, so that the display can be brought under test. Resolves #550
The template changes from `network-<CHAINID>` to `chain-<CHAINID>`. There is now also a validation test against the user provided deployment id to ensure it matches our rules. Specifically that it starts with an upper or lowercase letter, and then only includes alpha numerics, underscores or dashes. Resolves #551.
The blank line is also removed from status displays. Resolves #554.
Display a warning message in the `deploy` task. It indicates that deployment is against an ephemeral network. The implementation is to add extra information to the start run event, specifically the `deploymentDir`, which then allows the UI event handler to determine if the deployment is ephemeral. This means injecting that information down throught the `deploy`/`deployer` barrier. This is not ideal, but I think it is legitimate to assume the UI should have access to all the passed in config. Resolves #553.
Switch the `artifact` version of `contractAt` to: ```js m.contractAt(name, artifact, address) ``` This brings it into line with other artifact based signatures. Resolves #557.
* update the status text for success * update text of timed out futures * update the failed text * update status text for started futures * don't expose strategy level information This also updates the identifiers in the tests.
[WIP] Review error messages (PR 1)
Define call ids based on the entire id of the contract being called, not just its name (PR 2)
…messages Better validation error messages
Support fully qualified contract names (PR 3)
It splits the errors into two cases, with different messages:
- An autogenerated id is duplicated.
- An id provided by the user is duplicated.
It also changes the wording, so that we use the full id ("Module#autoId") only
in the autogenerated case, and just the id ("userProvidedId") when the user
provides it. This change is because the user may not recognize that
"Module#userProvidedId" refers to the string they provided, as it has
been changed.
Moves the `add future id to the builder` (used for checking duplicates across
modules and submodules) out from the assertion method and into each api
action. When a future is added to the module, we consider it safe to add
the future id to the list of previously seen future ids.
---------
Co-authored-by: John Kane <[email protected]>
Our display of the deployment starting panel is dependent on the `RUN_START` event. This event was being triggered only once for a deployment, on initialization of the deployment state, on the very first run of the deployment. The panel was not being displayed on subsequent runs. Several moves have been made to fix this: 1. The original run start has been renamed to `DEPLOYMENT_INITIALIZE` and only has responsibility for setting the `chainId`. It is only ever run as the first message in a new deployment. 2. The execution engine is no longer invoked if there are no batches to run (i.e. no uncompleted futures). 3. A new `RUN_START` that is UI only has been added to the deployer. If there is a batch to execute, and the execution engine is being invoked, we trigger the new `RUN_START`. 4. The deployment initialize event in the state reducer only sets the `chainId` (rather than clearing executionStates as well). The UI for displaying the execution panel has been shifted to the new run start event. These changes gives us the correct UI combinations: - Show Deploying panel on fresh deploy - Only show complete on rerun with no new futures - Show Deploying panel on rerun with new futures Fixes #566.
Add the reconciliation warnings to the deploying panel. Refactor the test format utility used in display tests as well. Fixes #567
Update the packages versions and changelogs for the `0.15.7 - 2024-10-24` release.
* fix: add additional validation for global parameters Default values for parameters masked the test cases for our global parameters feature. This commit adds additional tests for parameters without default values and enhances the validations to support Global Parameters as an option. * fix: use global params where other module params exist If there is a parameter defined in Global parameters, and a different parameter defined at the module level, we were failing to find the global parameter. This has been fixed with a more explicit lookup in `resolveModuleParameter`. Two tests have been added to check module param lookup where a different global parameter exists, and global parameter lookup where a different module parameter exists. * refactor: pull out shared validation code Pull out the shared validation resolution code for module/global/default value resolution for module parameters. Fixes #834
To ensure we work with nested parameters we use a replacer function on stringify, rather than swapping out individual params.
Update the packages versions and changelogs for the `0.15.8 - 2024-11-22` release.
Update the packages versions and changelogs for the `0.15.9 - 2024-12-18` release.
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is the initial migration of the https://github.com/NomicFoundation/hardhat-ignition repo into the hardhat monorepo. The commit history should be preserved from the original ignition repo, and the entire project should now live in this repo within
packages/hardhat-ignition.The ignition project is not yet functional within this repo, that will come in a follow-up PR. This PR is simply the merge of the commit histories.