GeoFlutterFire icon indicating copy to clipboard operation
GeoFlutterFire copied to clipboard

Move flutter_lints to dev_dependencies

Open jacksoncurrie opened this issue 2 years ago • 5 comments

Due to the flutter_lints package being in the dependencies many people will get version conflicts.

flutter_lints is only used in development so doesn't need to be shipped with this package. This will solve conflicts for all versions going forward and can be updated internally at any stage to fit project needs.

Moving to dev_dependencies: is also recommended by the flutter team when using flutter_lints as seen here.

jacksoncurrie avatar Jul 02 '22 12:07 jacksoncurrie

This works until 3.0.2. 3.0.3 is blocked on flutter_lints ^2.0.1

Because XXX depends on geoflutterfire ^3.0.3 which depends on flutter_lints ^1.0.0, flutter_lints ^1.0.0 is required.
So, because XXX depends on flutter_lints ^2.0.1, version solving failed.

wilpar avatar Aug 05 '22 18:08 wilpar

This works until 3.0.2. 3.0.3 is blocked on flutter_lints ^2.0.1


Because XXX depends on geoflutterfire ^3.0.3 which depends on flutter_lints ^1.0.0, flutter_lints ^1.0.0 is required.

So, because XXX depends on flutter_lints ^2.0.1, version solving failed.

But if we move it to dev_dependencies geoflutterfire will not depend on flutter_lints at all when used in projects.

jacksoncurrie avatar Aug 05 '22 20:08 jacksoncurrie

Same Issue Again So, because xx depends on flutter_lints ^2.0.1, version solving failed.

Abubakarharuna10 avatar Dec 12 '22 16:12 Abubakarharuna10

Same Issue Again So, because xx depends on flutter_lints ^2.0.1, version solving failed.

You'll want to see what depends on flutter_lints: ^2.0.1. This should always be in dev_dependencies, and only in your app. Any other dependencies should remove this from their dependency list.

jacksoncurrie avatar Dec 12 '22 19:12 jacksoncurrie

I noticed that this is also fixed in PR #197, so if that gets merged, this can be dismissed.

jacksoncurrie avatar Dec 12 '22 19:12 jacksoncurrie