-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Document breaking changes in the update pipeline #3769
Comments
@roji I didn't work on this when doing the 7.0 breaking changes because I'm not sure what needs doing where. |
Yeah, to be honest, I'm not sure what user-facing breaking changes we have here (apart from the trigger trouble which is already documented). The fact we no longer wrap single statements in a transaction may theoretically cause a behavior change in some scenarios, but I'd have to think hard to find such a scenario (we should maybe wait until a user hits one?). (There are certainly many provider-facing breaking changes here, but I'm not a fan of doing a breaking change note for those) |
Agreed. |
Maybe that EF Core runs too fast now? 🤣 |
Exactly. Luckily you have a "go slow" switch. |
Issues:
Since the breaking changes here are quite wide-reaching, reach out proactively to provider writers to help them.
The text was updated successfully, but these errors were encountered: