Merge pull request #420 from Workiva/generate-null-safe-run-script

FEDX-518 Now that dart_dev itself is null-safe, it should be able to generate a null-safe run script

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 4.0.3
+
+- Fix generation of the run script that the `dart_dev` CLI uses so that it can
+run with sound null safety (and thus run on Dart 3).
+
 ## 4.0.1
 
 - Fix type mismatch between the expected return type of the function passed to

lib/src/executable.dart

@@ -107,21 +107,13 @@ bool get shouldWriteRunScript =>
     !_runScript.existsSync() ||
     _runScript.readAsStringSync() != buildDartDevRunScriptContents();
 
-/// Whether dart_dev itself has opted into null-safety.
-const _isDartDevNullSafe = false;
-
 String buildDartDevRunScriptContents() {
   final hasCustomToolDevDart = File(paths.config).existsSync();
   // If the config has a dart version comment (e.g., if it opts out of null safety),
   // copy it over to the entrypoint so the program is run in that language version.
   var dartVersionComment = hasCustomToolDevDart
       ? getDartVersionComment(File(paths.config).readAsStringSync())
       : null;
-  // If dart_dev itself is not null-safe, opt the entrypoint out of null-safety
-  // so the entrypoint doesn't fail to run in packages that have opted into null-safety.
-  if (!_isDartDevNullSafe && dartVersionComment == null) {
-    dartVersionComment = '// @dart=2.9';
-  }
 
   return '''
 ${dartVersionComment ?? ''}