search

maplibre / flutter-maplibre-gl

Customizable, performant and vendor-free vector and raster maps, flutter wrapper for maplibre-native and maplibre-gl-js (fork of flutter-mapbox-gl/maps)
https://pub.dev/packages/maplibre_gl
Other
186 stars 106 forks source link

feat!: use lower camel case for enum fields and consts #415

Closed josxha closed 1 month ago

josxha commented 1 month ago

Lower camel case is the preferred naming convention for constants and enum fields. This pull request is breaking.

kuhnroyal commented 1 month ago

This can go into a 0.20.0 release. Is there a lint that we can enable for this?

josxha commented 1 month ago

Is there a lint that we can enable for this?

It might be possible to create decrecations for the old enum fields. Would require some temporary code though.

kuhnroyal commented 1 month ago

I think thats overkill but the disabled lint in analysis_options ca be enabled with this change?

josxha commented 1 month ago

Ohh you're right, totally forgot about that! Update: fixed