-
Notifications
You must be signed in to change notification settings - Fork 433
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
Fix CI tests #482
Fix CI tests #482
Conversation
…ardhat in smart_contract_tests to remove previous node version restrictions because of (now fixed) hardhat bug
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.
I think we should add some information about the current supported node.js versions. Currently, the info is: "First off, make sure you have a recent version of Node.js installed. While any version after v12 should work fine, we recommend you install v16 or later." However, it's important to explicitly specify that versions older than v16 are not guaranteed to work.
Maybe:
"Ensure you have a recent version of Node.js installed. Any version from v16 functions properly. Versions prior to v16 are not guaranteed to work."
@xavi-pinsach Thanks! I've added note that non-LTS versions are not guaranteed to work too. Does it sound right to you now? |
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.
Can you remove v16 from tutorial.yml?
on: [push, pull_request]
)