-
Notifications
You must be signed in to change notification settings - Fork 110
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
Automatic migration planning #1658
Conversation
pub prechecks: Vec<AutoMigratePrecheck<'def>>, | ||
/// The migration steps to perform. | ||
/// Order should not matter, as the steps are independent. | ||
pub steps: Vec<AutoMigrateStep<'def>>, |
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.
Is it guaranteed to not matter, or does it just happen to be that way because of the types of automigration we do 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.
This looks really good. Left a comment.
2cfd74d
to
6691e40
Compare
This now also fixes some stuff in ModuleDef introduced by miscommunication while collaborating with Noa |
Description of Changes
Implements planning, does not yet integrate.
API and ABI breaking changes
N/A
Expected complexity level and risk
0
Testing
See added tests.