Skip to content
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

docs: improve Hardhat plugin example using local path, intermediate contract and console.log #110

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

acuarica
Copy link
Contributor

@acuarica acuarica commented Nov 6, 2024

Description:

This PR fixes the Hardhat plugin example by installing the plugin using a local path (instead of a Git URL) to avoid installation problems. In addition, it improves example by adding a test that transferFrom using an intermediate contract. Finally, it demonstrates that console.log can be used as usual.

Related issue(s):

Fixes #109.

Notes for reviewer:

JSdoc for the getAddresses function was improved to references its source 7dc2619.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
Signed-off-by: Luis Mastrangelo <luis@swirldslabs.com>
@acuarica acuarica requested a review from a team as a code owner November 6, 2024 01:40
@acuarica acuarica linked an issue Nov 6, 2024 that may be closed by this pull request
@acuarica acuarica requested a review from stoyanov-st November 6, 2024 01:40
@acuarica acuarica self-assigned this Nov 6, 2024
@acuarica acuarica added the docs Improvements or additions to documentation label Nov 6, 2024
@acuarica acuarica merged commit d0502e4 into main Nov 6, 2024
11 checks passed
@acuarica acuarica deleted the 109-hardhat-improve-hardhat-plugin-example branch November 6, 2024 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Hardhat] Improve Hardhat plugin example
2 participants