-
Notifications
You must be signed in to change notification settings - Fork 177
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
editionupdate #2681
base: master
Are you sure you want to change the base?
editionupdate #2681
Conversation
Hi @pheobeayo, please check out my last comment in #2651 :) |
@franciszekjob Okay |
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.
Doesn't resolve the linked issue
@pheobeayo edition need to be changed in other packages as well, please verify it. If you have any questions, feel free to ask :). |
@franciszekjob Kindly check my last commits, please. I made changes in some other packages. Kindly point out others I didn't pick out if all the changes haven't been made |
@pheobeayo there are many more packages, e.g. under |
Hi @pheobeayo, please fix the errors on CI. |
Hi @pheobeayo do you need any help? |
@franciszekjob I need help fixing the errors on the CI, I have been trying to resolve it since but I haven't gotten any headway, any clue? |
Bumping edition results in occurring errors (which are visible on CI, example). Alternatively, you can open any package in which you bumped
Hope this helps you 😄 |
Okay, thank you! |
Closes #
Introduced changes
-migrate scarb package to edition 2024_07
Checklist
edition = "2024_07"
#2651 ] Linked relevant issueCHANGELOG.md