Mateus Felipe C. C. Pinto

Results 262 comments of Mateus Felipe C. C. Pinto

If there's any interest in reviewing a PR for this, I may do it.

@srawlins Did you get the chance to talk about it in the analyzer team meeting?

This is a hard blocker for me. I have a similar case to what @springfielddatarecovery. Consider a `history` table, with this structure: | id | date | person | status...

Oh, I got this recently, I lost some hours trying to understand why the `minimumSize` was not being respect before realizing that it should be a bug in Flutter itself.

@kevmoo What would be the failure cases for this PR? Can you give an example?

> > @kevmoo What would be the failure cases for this PR? Can you give an example? > > Unsupported types in the set is the big one. `null` isn't...

> You'll need to rebase this fix. I rebased to master and included the requested test. In my local machine, a test was failing, but I didn't change anything related...

Mmm, I had made a regular merge, but GitHub is saying to me that "this branch must not contain merge commits", so I rewrote the commits using a rebase instead,...

Well, the test that did not pass locally passed here, so we are good with it.

@kevmoo Is there something preventing the code review from proceeding?