-
Notifications
You must be signed in to change notification settings - Fork 488
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 in hardhat.config.js file #99
Comments
HI @luislucena16! Can you please link to the file you're referring to? thanks! |
hi @zeuslawyer! Sure, https://github.com/smartcontractkit/hardhat-starter-kit/blob/main/hardhat.config.js line 100. thank you for your response! |
@luislucena16 - no problem! I believe what you're saying makes sense. Do you want to submit a PR? In case you've never submitted one, you can fork this repo (it will be called the "upstream" repo) and then you can follow this process: Let me know if you want to try that :) |
yes of course, I will open the PR and make the changes! thanks :) @zeuslawyer |
hey @zeuslawyer I have already opened the PR, here is the link #100 |
thanks @luislucena16 . PR approved. Appreciate your help! |
hey @PatrickAlphaC I have a doubt about line 100, it is calling the RandomNumberConsumer but I should assign the V2 at the end of the name, please correct me if I am wrong.
The text was updated successfully, but these errors were encountered: