-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
mysql create table add table and colunm comment #622
Conversation
# Conflicts: # src/table/column.rs # src/table/create.rs
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.
Hey @Caisin, thanks for contributing!! Some comments for you :P
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.
@Caisin thank you for the PR! Small comments.
Co-authored-by: Ivan Krivosheev <py.krivosheev@gmail.com>
use by sea-orm codegen
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.
@Caisin can you move part with rbatist into other PR?
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.
@CaisinThank you! LGTM!
@billy1624 do you want to include it into 0.29 release?) |
src/backend/mysql/table.rs
Outdated
let comment = comment.replace('\'', "\\'"); | ||
write!(sql, "COMMENT '{comment}'").unwrap() |
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.
We have a standard way to escape string
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.
Thank you for the contribution
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.
done
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.
Thank you again for your contribution.
/// r#"`character` varchar(255) NOT NULL,"#, | ||
/// r#"`size_w` int NOT NULL,"#, | ||
/// r#"`size_h` int NOT NULL,"#, | ||
/// r#"`font_id` int DEFAULT NULL,"#, | ||
/// r#"CONSTRAINT `FK_2e303c3a712662f1fc2a4d0aad6`"#, | ||
/// r#"FOREIGN KEY (`font_id`) REFERENCES `font` (`id`)"#, | ||
/// r#"ON DELETE CASCADE ON UPDATE CASCADE"#, | ||
/// r#") COMMENT 'table comment'"#, | ||
/// r#") COMMENT 'table\'s comment'"#, |
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 like the extra '
sea-orm 0.11.2 still not working with comment ,is this feature published now ? |
No, it's been merged, but not yet released. |
🎉 Released In 0.29.1 🎉Thank you everyone for the contribution! |
New Features
mysql create table add table and colunm comment
Changes