Skip to content

Commit ecc4ef3

Browse files
authored
Review and update check imports script (#9660)
1 parent 3ef42c1 commit ecc4ef3

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

scripts/check_imports.swift

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -38,12 +38,9 @@ let skipDirPatterns = ["/Sample/", "/Pods/", "FirebaseStorageInternal/Tests/Inte
3838

3939
// The following are temporary skips pending working through a first pass of the repo:
4040
[
41-
"FirebaseAppDistribution",
42-
"FirebaseCore/Extension", // TODO(v9): work through adding this back.
43-
"Firebase/CoreDiagnostics",
41+
"Firebase/CoreDiagnostics/FIRCDLibrary/Protogen/nanopb",
4442
"FirebaseDatabase/Sources/third_party/Wrap-leveldb", // Pending SwiftPM for leveldb.
4543
"Example",
46-
"FirebaseInstallations/Source/Tests/Unit/",
4744
"Firestore",
4845
"GoogleUtilitiesComponents",
4946
"FirebasePerformance/ProtoSupport/",
@@ -77,13 +74,11 @@ private func checkFile(_ file: String, logger: ErrorLogger, inRepo repoURL: URL)
7774
// Not a source file, give up and return.
7875
return
7976
}
80-
let isPublic = file.range(of: "/Public/") != nil &&
81-
// TODO: Skip legacy GDTCCTLibrary file that isn't Public and should be moved.
82-
file.range(of: "GDTCCTLibrary/Public/GDTCOREvent+GDTCCTSupport.h") == nil
77+
let isPublic = file.range(of: "/Public/") != nil
8378
let isPrivate = file.range(of: "/Sources/Private/") != nil ||
8479
// Delete when FirebaseInstallations fixes directory structure.
8580
file.range(of: "Source/Library/Private/FirebaseInstallationsInternal.h") != nil ||
86-
file.range(of: "GDTCORLibrary/Internal/GoogleDataTransportInternal.h") != nil
81+
file.range(of: "FirebaseCore/Extension") != nil
8782

8883
// Treat all files with names finishing on "Test" or "Tests" as files with tests.
8984
let isTestFile = file.contains("Test.m") || file.contains("Tests.m") ||

0 commit comments

Comments
 (0)