fix: linting and analyzer issues

fluttercommunity / flutter_launcher_icons

Flutter Launcher Icons - A package which simplifies the task of updating your Flutter app's launcher icon. Fully flexible, allowing you to choose what platform you wish to update the launcher icon for and if you want, the option to keep your old launcher icon in case you want to revert back sometime in the future. Maintainer: @MarkOSullivan94

https://pub.dev/packages/flutter_launcher_icons

MIT License

1.99k stars 396 forks source link

fix: linting and analyzer issues #533

Open OutdatedGuy opened 9 months ago

OutdatedGuy commented 9 months ago

Fixes

Analyzer uses deprecated options
Missing required trailing commas

New

Strictly mention data types

OutdatedGuy commented 9 months ago

Should Map<String, dynamic> not work as well as Map<dynamic, dynamic>?

Rather be more specific with our types if we can

Yeah true, my bad. I'll change it.

OutdatedGuy commented 8 months ago

Hi @MarkOSullivan94, I have made types more specific wherever possible without having an error. Please review it.

OutdatedGuy commented 5 months ago

@MarkOSullivan94, when would this be merged. A ton of issues and PRs are pilling up for this package.

Is there anything I can help with?