-
Notifications
You must be signed in to change notification settings - Fork 192
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
Change ForeignKeyAction to enum in the generator #359
Change ForeignKeyAction to enum in the generator #359
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #359 +/- ##
===========================================
+ Coverage 82.83% 83.05% +0.22%
===========================================
Files 68 68
Lines 1660 1682 +22
===========================================
+ Hits 1375 1397 +22
Misses 285 285
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
I put the extension on |
… onConflict values
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.
Awesome! Thanks for making our codebase better 🙌
… onConflict values
Also fix formatting from rebase
50855a3
to
c5ebdf9
Compare
…-foreignkeyaction-enum # Conflicts: # floor_generator/lib/misc/extension/dart_object_extension.dart # floor_generator/lib/processor/entity_processor.dart
@mqus I'll look into getting this one merged if you don't mind? |
Thank you! |
* Change ForeignKeyAction to enum in the generator (#359) * Added support for WITH statements for DatabaseViews (#443) * Documentation update on DateTimeConverter sample (#442) * Make foreign key mapping null safe Co-authored-by: mqus <8398165+mqus@users.noreply.github.com> Co-authored-by: themadmrj <themadmrj@users.noreply.github.com> Co-authored-by: Cássio Seffrin <cassioseffrin@gmail.com>
This changes ForeignKeyAction to be an enum instead of a string. This allows us to be more strict when processing the annotation as well as be more confident when dealing with annotations.
This is a preparation for the sqlanalyzer, to avoid matching strings when converting floor entities to moor/sqlparser tables.