Fix SqlConstantExpression obsolete constructor #35357
Conversation
There was a problem hiding this comment.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
If the API is going removed in EFCore 10.x, this change might make more sense as a minor fix to a 9.x.y (to ease the upgrade path by warning users while preserving the old behavior) than in the main branch... but I guess there is no need to hurry getting rid of the obsolete API if it is working 😉 Thank you for the immediate response! |
|
We generally don't remove obsolete APIs in odd versions (short-term releases), since many users upgrade from LTS to LTS. So for someone upgrading from 8.0 to 10.0, obsoleting in 9.0 and removing in 10.0 would be a bit like immediate removal... But we'll most probably be removing this in 11.0. |
Forgoing a test since this fix is for an obsolete constructor that will go away soon anyway.
Fixes #35355
/cc @ranma42