Use "flutter[.bat]" instead of "pub[.bat]" for flutter packages (#59, #45) (#318)

Use "flutter[.bat]" instead of "pub[.bat]" for flutter packages (#59, #45)

Author: BenVercammen

mono_repo/lib/src/ci_test_script.dart

@@ -16,12 +16,32 @@ function $commandName() {
   fi
 }''';
 
+String _dartCommandContentPub(String commandName) => '''
+# When it is a flutter repo (check the pubspec.yaml for "sdk: flutter")
+# then "flutter" is called instead of "pub".
+# This assumes that the Flutter SDK has been installed in a previous step.
+function $commandName() {
+  if grep -Fq "sdk: flutter" "\${PWD}/pubspec.yaml"; then
+    if [[ \$TRAVIS_OS_NAME == "windows" ]]; then
+      command flutter.bat pub "\$@"
+    else
+      command flutter pub "\$@"
+    fi
+  else
+    if [[ \$TRAVIS_OS_NAME == "windows" ]]; then
+      command pub.bat "\$@"
+    else
+      command pub "\$@"
+    fi
+  fi
+}''';
+
 final bashScriptHeader = '''
 #!/bin/bash
 $createdWith
 
 # Support built in commands on windows out of the box.
-${_dartCommandContent('pub')}
+${_dartCommandContentPub('pub')}
 ${_dartCommandContent('dartfmt')}
 ${_dartCommandContent('dartanalyzer')}''';
 

mono_repo/test/script_integration_outputs/readme_ci.txt

@@ -2,11 +2,22 @@
 # Created with package:mono_repo v1.2.3
 
 # Support built in commands on windows out of the box.
+# When it is a flutter repo (check the pubspec.yaml for "sdk: flutter")
+# then "flutter" is called instead of "pub".
+# This assumes that the Flutter SDK has been installed in a previous step.
 function pub() {
-  if [[ $TRAVIS_OS_NAME == "windows" ]]; then
-    command pub.bat "$@"
+  if grep -Fq "sdk: flutter" "${PWD}/pubspec.yaml"; then
+    if [[ $TRAVIS_OS_NAME == "windows" ]]; then
+      command flutter.bat pub "$@"
+    else
+      command flutter pub "$@"
+    fi
   else
-    command pub "$@"
+    if [[ $TRAVIS_OS_NAME == "windows" ]]; then
+      command pub.bat "$@"
+    else
+      command pub "$@"
+    fi
   fi
 }
 function dartfmt() {

tool/ci.sh

@@ -2,11 +2,22 @@
 # Created with package:mono_repo v4.0.0-dev
 
 # Support built in commands on windows out of the box.
+# When it is a flutter repo (check the pubspec.yaml for "sdk: flutter")
+# then "flutter" is called instead of "pub".
+# This assumes that the Flutter SDK has been installed in a previous step.
 function pub() {
-  if [[ $TRAVIS_OS_NAME == "windows" ]]; then
-    command pub.bat "$@"
+  if grep -Fq "sdk: flutter" "${PWD}/pubspec.yaml"; then
+    if [[ $TRAVIS_OS_NAME == "windows" ]]; then
+      command flutter.bat pub "$@"
+    else
+      command flutter pub "$@"
+    fi
   else
-    command pub "$@"
+    if [[ $TRAVIS_OS_NAME == "windows" ]]; then
+      command pub.bat "$@"
+    else
+      command pub "$@"
+    fi
   fi
 }
 function dartfmt() {