Conversation
ales-otf
left a comment
There was a problem hiding this comment.
Finally we can have it in the main repo! And having it type-safe is an additional benefit. I noticed a few code quality issues though. Additionally to them there are a lot of commented code. But feel free to ignore my comments.
ales-otf
left a comment
There was a problem hiding this comment.
thanks Junius! i've noticed just a few more issues in readme. but they are minor, you can ignore them if you wish.
evm-tests/README.md
Outdated
|
|
||
| npm update @polkadot-api/descriptors | ||
| ```bash | ||
| yarn upgrade @polkadot-api/descriptors |
There was a problem hiding this comment.
missed this last time and confused now. as far as i understand it, this command does the same as cargo update in rust - it updates dependencies versions in the manifest. i suppose we wouldn't like it to be happened every time the repo is cloned. otherwise why this command is listed here particularly for this dependency?
There was a problem hiding this comment.
I just found out if we generate a new metadata in .papi which including the new extrinsic, for instance. But the IDE doesn't show the new one. I need run it to refresh. It is little confused. I will remove it and put it in personal notes.
There was a problem hiding this comment.
oh, if it's needed to generate metadata, then it's better to keep it, but mentioned it in here.
Description
The PR add e2e test cases for EVM in subtensor. All are implemented in type script.
Related Issue(s)
Type of Change
Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist
cargo fmtandcargo clippyto ensure my code is formatted and linted correctlyScreenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.