-
-
Notifications
You must be signed in to change notification settings - Fork 38
Conversation
I only have flutter channel beta and stable installed. Looks like the formatter changes in the latest dart release. |
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.
@Vinzent03 Sorry for taking my time to review this one!
Thank you so much for the quick fix here! I just had few small questions, but it all looks amazing!
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.
Looks amazing to me! Looks like the failing tests is just the formatting rule changing in dev branch of dart, so let's merge it in!
Could we update the package version number, changelog, and version.dart as well?
@dshukertjr I will fix supabase/supabase-flutter/issues/201 today as well, so please wait with release. |
Cool, merging this one then! |
There were multiple problems:
hint
was missingdata
was not null, but an empty string?PostgrestResponse
instead ofdata
directly, becausecount
was not null.fix supabase/supabase-flutter/issues/195