feat: add hardhat plugin to support HTS emulation in fork testing #50
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
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
arianejasuwienas
requested review from
rbarker-dev,
leninmehedy and
stoyanov-st
arianejasuwienas
added
the
New Feature
acuarica
requested changes
Sep 25, 2024
There was a problem hiding this comment.
@arianejasuwienas great to have this. I see there are leftovers from the Hardhat template, can we remove all of these?
acuarica
requested changes
Sep 25, 2024
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Nana-EC
requested changes
Sep 26, 2024
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
acuarica
requested changes
Sep 30, 2024
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
…thing more to be here Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
acuarica
reviewed
Oct 2, 2024
arianejasuwienas
commented
Oct 3, 2024
2 tasks
|
@arianejasuwienas see if you can fix DCO error https://github.com/hashgraph/hedera-forking/pull/50/checks?check_run_id=31028928684 after merging the PR. Moreover, please add a description for this PR and title. |
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
arianejasuwienas
force-pushed
the
44-hardhat-plugin
branch
from
24f048c to
923b122
Compare
|
I've fixed it, but I'm not sure if that was the smartest way to do it (rebase, reset head , commit again), since now I'm the author of your changes as well. But since we will lose the commit history anyway when merging the PR perhaps this approach is good enough? Let me know @acuarica . |
|
@arianejasuwienas yes, that's good enough, thanks for the heads up and for taking care of this. |
acuarica
reviewed
Oct 3, 2024
acuarica
reviewed
Oct 3, 2024
acuarica
reviewed
Oct 3, 2024
…ed (for current config options) file. Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
Signed-off-by: Mariusz Jasuwienas <mariusz.jasuwienas@arianelabs.com>
arianejasuwienas
changed the title
arianejasuwienas
changed the title
acuarica
added
feature
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.
Description:
Hardhat plugin allowing users to test HTS on the forked networks . Will provide developers with the possibility to use the custom hardhat provider that will preset the precompile byte code on the network as well as fill the storage slots related to the currently used tokens.
Related issue(s):
#44
Notes for reviewer:
Checklist