update hardhat #1190
update hardhat #1190
Conversation
| }, | ||
| polkadotHubTestnet: { | ||
| polkavm: true, | ||
| polkadot: true, |
There was a problem hiding this comment.
|
Hi @bee344! I'll temporarily (and softly) block this PR until we get this resolved: Looping in @UtkarshBhardwaj007 because, based on our last call, I understand we agreed on having docs for the incoming hackathon, but those were supposed to be added to the survival guide, not the official docs, right? Edit: or maybe this has nothing to do with the hackathon yet? and @bee344 wanted to create a hot-fix due to the hardhat situation? |
|
@nhussein11 yes, it's linked to the hackathon. Besides that, the docs are very outdated since there have been a lot of changes lately, so I don't see the harm in updating them to reflect the current state of the plugin. It does take into account a future release (0.2.0-pre6), but the only change that comes with that is an updated readme and updated projects generated by the CLI tool. |
| --8<-- 'code/develop/smart-contracts/dev-environments/hardhat/hardhat.config.js:6:7' | ||
| ... | ||
| --8<-- 'code/develop/smart-contracts/dev-environments/hardhat/hardhat.config.js:12:24' |
| 1. Update the Hardhat configuration file to add the local network as a target for local deployment: | ||
| ```javascript title="hardhat.config.js" hl_lines="13-16" |
| 4. Update your Hardhat configuration file with network settings for the Polkadot network you want to target: | ||
| ```javascript title="hardhat.config.js" hl_lines="18-22" |
There was a problem hiding this comment.
Check line numbers last time :-)
Hey @nhussein11 we did plan to do this directly in the main docs as discussed in the meeting. We will not do this in the beta docs because we don't want to share that externally yet. And also as Beto said this documentation was pretty dated anyway. |
|
@UtkarshBhardwaj007, ah, I understood that this update was for the hackathon survival guide, but in anyt case I think we should have some content for the hackathon somewhere (it doesn't really matter where). Thanks! |
nhussein11
left a comment
nhussein11
left a comment
There was a problem hiding this comment.
@bee344 facing this issue when reviewing the PR
|
|
||
| ```bash | ||
| npm install --save-dev @parity/hardhat-polkadot@latest | ||
| npm install --save-dev @parity/hardhat-polkadot@0.2.0-pre6 |
There was a problem hiding this comment.
That now needs to be changed to 0.2.0, right?
|
|
||
| ```bash | ||
| npm install --save-dev @parity/hardhat-polkadot@0.1.9 | ||
| npm install --save-dev @parity/hardhat-polkadot@0.2.0-pre6 |
There was a problem hiding this comment.
That now needs to be changed to 0.2.0, right?
|
Closed in favor of #1227 |
4 participants
📝 Description
Updated hardhat docs with latest functionality. Links are broken until a new hardhat release is done on Monday. Functionality is already present on
pre5, with the only difference being the sample project config.🔍 Review Preference
Choose one:
🤖 AI-Ready Docs
If content changed, regenerate AI files:
python3 scripts/generate_llms.py✅ Checklist