-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat: add tool-versions to scripts and contracts #112
base: main
Are you sure you want to change the base?
Conversation
Hi @coxmars and @danielcdz here is my pull request, thank you if you let me know what you think. I remain attentive to any changes I have to make. Excellent day bosses. π«‘π¨π»βπ» |
cairo/scripts/.tool-versions
Outdated
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.
This one can be deleted
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.
Done βοΈπ«‘
starknet/contracts/.tool-versions
Outdated
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.
This one can be deleted
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.
Done βοΈπ«‘
starknet/modules/.tool-versions
Outdated
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.
This one can be deleted
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.
Done βοΈπ«‘
@salazarsebas did you make sure to add the correct versions in the files? |
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.
LGTM, I already tested the versions in the scripts and others and everything works. Just apply Daniel changes. π«‘
I made the changes, please let me know what you think. Thanks for the feedback. |
@salazarsebas Once you finished applying the requested changes please click the round button here: |
Done, an apology for not doing it sooner. |
π Pull Request Title
Add the .tool-versions file to all relevant scripts and contracts directories that were not there before.
π οΈ Issue
π Description
This pul request ensures that the
.tool-versions
file is added to all directories containing scripts and contract files where it was previously missing. This ensures consistency across the development environment by specifying the exact versions of the tools required for the project in all relevant locations. This addition helps avoid compatibility issues and ensures that developers working on the project use the same tool versions across these directories.π Additional Notes