-
-
Notifications
You must be signed in to change notification settings - Fork 195
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
String parameter should take generic Into<String>
#594
Conversation
@billy1624 hello! I have pushed new commit. |
Hey @ikrivosheev, sorry for the delay. Had been starting a new proprietary project in the past two weeks loll. Now, I'm back :) |
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.
@billy1624 hello! LGTM!
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.
I think this has a real benefit - if given a String, we don't have to allocate a second time.
Yes, we only |
🎉 Released In 0.29.0 🎉Your participation is what makes us unique; your adoption is what drives us forward. |
🎉 Released In 0.29.1 🎉Thank you everyone for the contribution! |
PR Info
Into<String>
sea-orm#1439Breaking Changes
Changed the
&str
parameter of the following methods to a genericT: Into<String>
:Expr::cust()
:Expr::cust_with_values()
Expr::cust_with_expr()
Expr::cust_with_exprs()
TableForeignKey::name()
ForeignKeyCreateStatement::name()
ForeignKeyDropStatement::name()
TableIndex::name()
IndexCreateStatement::name()
IndexDropStatement::name()
SqlWriterValues::new()
ColumnType::custom()
TableCreateStatement::engine()
TableCreateStatement::collate()
TableCreateStatement::character_set()
TableRef::new()
LikeExpr::str()
Changed the
String
parameter of the following methods to a genericColumnType::extra()
:LikeExpr::new()
Changes
IntoLikeExpr
implementations for all types implementedInto<String>
Deprecates
LikeExpr::str()
, please useLikeExpr::new()
instead