Don't check licenses of generated files

dart-lang · Dec 12, 2024 · d9ba42f · d9ba42f
1 parent b25118a
commit d9ba42f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/pkgs/firehose/lib/src/health/license.dart b/pkgs/firehose/lib/src/health/license.dart
@@ -26,8 +26,8 @@ Future<List<String>> getFilesWithoutLicenses(
         if (ignoredFiles.none((glob) =>
             glob.matches(path.relative(file.path, from: repositoryDir.path)))) {
           var fileContents = File(file.path).readAsStringSync();
-          var fileContainsCopyright = fileContents.contains('// Copyright (c)');
-          if (!fileContainsCopyright) {
+          if (!fileIsGenerated(fileContents) &&
+              !fileContainsCopyright(fileContents)) {
             print(relativePath);
             return relativePath;
           }
@@ -40,3 +40,11 @@ Future<List<String>> getFilesWithoutLicenses(
 Done, found ${filesWithoutLicenses.length} files without license headers''');
   return filesWithoutLicenses;
 }
+
+bool fileIsGenerated(String fileContents) => fileContents
+    .split('\n')
+    .takeWhile((line) => line.startsWith('//') || line.isEmpty)
+    .any((line) => line.toLowerCase().contains('generate'));
+
+bool fileContainsCopyright(String fileContents) =>
+    fileContents.contains('// Copyright (c)');
diff --git a/pkgs/firehose/test_data/test_repo/pkgs/package2/lib/anotherLib.dart b/pkgs/firehose/test_data/test_repo/pkgs/package2/lib/anotherLib.dart
@@ -1,3 +1,6 @@
+// SOME COMMENT
+// THIS IS A GENERATED FILE
+
 int calculateUnused() {
   return 6 * 7;
 }