Ensure flutter format and fix offending files (#80)

The current Travis script will only print offending files, but won’t fail the build. Added `--set-exit-if-changed` to fix that.

Also, simplified the call from using `grep` and `xargs` to simple `flutter format .`, which recurses directories.

Lastly, fixed offending files that needed `dartfmt`. (But not in the directories that aren’t covered by `travis_script.sh`.
pull/85/head
Filip Hracek 6 years ago committed by GitHub
parent fbf40c0cd2
commit fa356540c6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -224,7 +224,6 @@ class PlaceMapState extends State<PlaceMap> {
void _confirmAddPlace(BuildContext context) async { void _confirmAddPlace(BuildContext context) async {
if (_pendingMarker != null) { if (_pendingMarker != null) {
// Create a new Place and map it to the marker we just added. // Create a new Place and map it to the marker we just added.
final Place newPlace = Place( final Place newPlace = Place(
id: Uuid().v1(), id: Uuid().v1(),

@ -18,7 +18,7 @@ do
../flutter/bin/flutter analyze ../flutter/bin/flutter analyze
# Run the formatter on all the dart files to make sure everything's linted. # Run the formatter on all the dart files to make sure everything's linted.
find . | grep "\.dart$" | xargs ../flutter/bin/flutter format -n ../flutter/bin/flutter format -n --set-exit-if-changed .
# Run the actual tests. # Run the actual tests.
../flutter/bin/flutter test ../flutter/bin/flutter test

@ -13,11 +13,8 @@ import 'package:veggieseasons/styles.dart';
import 'package:veggieseasons/widgets/veggie_card.dart'; import 'package:veggieseasons/widgets/veggie_card.dart';
class ListScreen extends StatelessWidget { class ListScreen extends StatelessWidget {
List<Widget> _generateVeggieRows( List<Widget> _generateVeggieRows(List<Veggie> veggies, Preferences prefs,
List<Veggie> veggies, {bool inSeason = true}) {
Preferences prefs,
{ bool inSeason = true }
) {
final cards = List<Widget>(); final cards = List<Widget>();
for (Veggie veggie in veggies) { for (Veggie veggie in veggies) {
@ -27,7 +24,8 @@ class ListScreen extends StatelessWidget {
future: prefs.preferredCategories, future: prefs.preferredCategories,
builder: (context, snapshot) { builder: (context, snapshot) {
final data = snapshot.data ?? Set<VeggieCategory>(); final data = snapshot.data ?? Set<VeggieCategory>();
return VeggieCard(veggie, inSeason, data.contains(veggie.category)); return VeggieCard(
veggie, inSeason, data.contains(veggie.category));
}), }),
)); ));
} }
@ -69,7 +67,8 @@ class ListScreen extends StatelessWidget {
), ),
); );
rows.addAll(_generateVeggieRows(appState.unavailableVeggies, prefs, inSeason: false)); rows.addAll(_generateVeggieRows(appState.unavailableVeggies, prefs,
inSeason: false));
return DecoratedBox( return DecoratedBox(
decoration: BoxDecoration(color: Color(0xffffffff)), decoration: BoxDecoration(color: Color(0xffffffff)),

@ -273,5 +273,4 @@ abstract class Styles {
static const ColorFilter desaturatedColorFilter = static const ColorFilter desaturatedColorFilter =
// 222222 is a random color that has low color saturation. // 222222 is a random color that has low color saturation.
ColorFilter.mode(Color(0xFF222222), BlendMode.saturation); ColorFilter.mode(Color(0xFF222222), BlendMode.saturation);
} }

@ -130,9 +130,8 @@ class VeggieCard extends StatelessWidget {
decoration: BoxDecoration( decoration: BoxDecoration(
image: DecorationImage( image: DecorationImage(
fit: BoxFit.cover, fit: BoxFit.cover,
colorFilter: isInSeason colorFilter:
? null isInSeason ? null : Styles.desaturatedColorFilter,
: Styles.desaturatedColorFilter,
image: AssetImage( image: AssetImage(
veggie.imageAssetPath, veggie.imageAssetPath,
), ),

Loading…
Cancel
Save