-
Notifications
You must be signed in to change notification settings - Fork 74
[Hardhat] Create compile, deploy, install and config pages #1150
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
base: staging/product-ia
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR splits a single Hardhat documentation page into three separate pages focused on installation/configuration, compilation/testing, and deployment. This improves documentation organization by separating concerns and making each topic more accessible.
Key changes:
- Created install-and-config.md covering Hardhat setup, prerequisites, and plugin installation
- Created compile-and-test.md covering contract compilation, testing environment setup, and test execution
- Created deploy-a-contract.md covering local and testnet deployment procedures
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| smart-contracts/dev-environments/hardhat/install-and-config.md | New page documenting Hardhat installation, configuration, and initial setup for Polkadot Hub development |
| smart-contracts/dev-environments/hardhat/compile-and-test.md | New page documenting contract compilation process and testing environment setup |
| smart-contracts/dev-environments/hardhat/deploy-a-contract.md | New page documenting contract deployment to local and live networks |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
- Update install-and-config.md to use standard Hardhat toolbox - Replace hardhat-polkadot init with standard hardhat init - Remove plugin upgrade section and PolkaVM-specific references - Update compile-and-test.md to use solc instead of resolc - Replace artifacts-pvm directory references with artifacts - Remove Substrate node and ETH-RPC adapter configuration - Update deploy-a-contract.md with inline standard Hardhat config - Delete obsolete hardhat-polkadot config snippet files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.snippets/code/smart-contracts/dev-environments/hardhat/get-started/hardhat-node-output.html
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
nhussein11
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Left some comments, feel free to ping me if anything
|
|
||
| ```bash | ||
| npx hardhat run scripts/interact.js --network polkadotHubLocal | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Attach terminal output here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to be attached as soon as the local node is deployed from master
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
Co-authored-by: Nicolás Hussein <80422357+nhussein11@users.noreply.github.com>
nhussein11
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! some minor details but thank you very much @kapetan3sid !!
This PR introduces 3 pages:
These pages were split from
develop/smart-contracts/dev-environments/hardhat.mdinmasterbranch