Tim Maffett
Tim Maffett
@werainkhatri i addressed the issues and it is now green
@darrenaustin I might mention that the testing code that I added for each of the Chip classes is a little different then the test added to `ChoiceChip` by https://github.com/flutter/flutter/pull/110908 I...
> very sorry for the delayed review. life just got busy. > > this LGTM module some minor nits (will commit them myself via github) and the unresolved conversation above....
Hi @werainkhatri - ok, I think I have removed the redundant material test you are talking about. I now just check to verify that the `customIconTheme` made it to the...
@TahaTesser I have made the requested changes. Please take a look when you have a chance.
@TahaTesser Not sure how I missed that! Should be fixed now.
The test check failures do not seem to be related to this pr? Updating the branch and giving it another go.
@timmaffett The conflicts should now be resolved.
@TahaTesser I added those const the analyzer was complaining about.