Switch to glob

.github/workflows/health.yaml

@@ -78,11 +78,14 @@ on:
         type: boolean
         required: false
       use-flutter:
-        description: >-
-          Whether to setup Flutter in this workflow.
+        description: Whether to setup Flutter in this workflow.
         default: false
         required: false
         type: boolean
+      ignore_license:
+        description: Which files to ignore when checking for licenses.
+        required: false
+        type: string
 
 jobs:
   version:
@@ -117,6 +120,7 @@ jobs:
       warn_on: ${{ inputs.warn_on }}
       local_debug: ${{ inputs.local_debug }}
       use-flutter: ${{ inputs.use-flutter }}
+      ignore: ${{ inputs.ignore_license }}
 
   coverage:
     if: contains(${{ inputs.checks }}, 'coverage')

.github/workflows/health_base.yaml

@@ -51,6 +51,7 @@ on:
         type: boolean
       ignore:
         description: The files to ignore for this check.
+        default: ""
         required: false
         type: string
 

.github/workflows/health_internal.yaml

@@ -15,3 +15,4 @@ jobs:
       checks: version,changelog,license,coverage,breaking,do-not-submit
       fail_on: version,changelog,do-not-submit
       warn_on: license,coverage,breaking
+      ignore_license: '**.g.dart'

pkgs/firehose/lib/src/health/health.dart

@@ -9,6 +9,7 @@ import 'dart:io';
 import 'dart:math';
 
 import 'package:collection/collection.dart';
+import 'package:glob/glob.dart';
 import 'package:path/path.dart' as path;
 import 'package:pub_semver/pub_semver.dart';
 
@@ -53,14 +54,14 @@ class Health {
     this.failOn,
     this.coverageweb,
     List<String> ignored,
-  ) : ignoredFiles = ignored.map(RegExp.new).toList();
+  ) : ignoredFiles = ignored.map((e) => Glob(e)).toList();
   final github = GithubApi();
 
   final String check;
   final List<String> warnOn;
   final List<String> failOn;
   final bool coverageweb;
-  final List<RegExp> ignoredFiles;
+  final List<Glob> ignoredFiles;
 
   Future<void> healthCheck() async {
     // Do basic validation of our expected env var.

pkgs/firehose/lib/src/health/license.dart

@@ -5,6 +5,7 @@
 import 'dart:io';
 
 import 'package:collection/collection.dart';
+import 'package:glob/glob.dart';
 import 'package:path/path.dart' as path;
 
 final license = '''
@@ -13,7 +14,7 @@ final license = '''
 // BSD-style license that can be found in the LICENSE file.''';
 
 Future<List<String>> getFilesWithoutLicenses(
-    Directory repositoryDir, List<RegExp> ignoredFiles) async {
+    Directory repositoryDir, List<Glob> ignoredFiles) async {
   var dartFiles = await repositoryDir
       .list(recursive: true)
       .where((f) => f.path.endsWith('.dart'))
@@ -26,7 +27,7 @@ Future<List<String>> getFilesWithoutLicenses(
         if (!fileContainsCopyright) {
           var relativePath =
               path.relative(file.path, from: Directory.current.path);
-          if (ignoredFiles.none((regex) => regex.hasMatch(relativePath))) {
+          if (ignoredFiles.none((regex) => regex.matches(relativePath))) {
             print(relativePath);
             return relativePath;
           }

pkgs/firehose/pubspec.yaml

@@ -19,6 +19,7 @@ dependencies:
   pub_semver: ^2.1.0
   pubspec_parse: ^1.2.3
   yaml: ^3.1.0
+  glob: ^2.1.2
 
 dev_dependencies:
   dart_flutter_team_lints: ^2.0.0