Skip to content

Commit d16ad20

Browse files
Merge pull request #3076 from SwiftPackageIndex/issue-3074-alerting-false-positives
Reduce chance of false positives in alerting
2 parents 8f26223 + 635731f commit d16ad20

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

Sources/App/Commands/Alerting.swift

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -98,11 +98,11 @@ extension Alerting {
9898
builds.validateBuildsPresent().log(check: "CHECK_BUILDS_PRESENT")
9999
builds.validatePlatformsPresent().log(check: "CHECK_BUILDS_PLATFORMS_PRESENT")
100100
builds.validateSwiftVersionsPresent().log(check: "CHECK_BUILDS_SWIFT_VERSIONS_PRESENT")
101-
builds.validatePlatformsSuccessful().log(check: "CHECK_BUILDS_PLATFORMS_SUCCESSFUL")
102-
builds.validateSwiftVersionsSuccessful().log(check: "CHECK_BUILDS_SWIFT_VERSIONS_SUCCESSFUL")
103101
builds.validateRunnerIdsPresent().log(check: "CHECK_BUILDS_RUNNER_IDS_PRESENT")
104-
builds.validateRunnerIdsSuccessful().log(check: "CHECK_BUILDS_RUNNER_IDS_SUCCESSFUL")
105-
if builds.count >= 1000 { // only run this test if we have a decent number of builds
102+
if builds.count >= 1000 { // only run these tests if we have a decent number of builds, to reduce chance of false positives
103+
builds.validatePlatformsSuccessful().log(check: "CHECK_BUILDS_PLATFORMS_SUCCESSFUL")
104+
builds.validateSwiftVersionsSuccessful().log(check: "CHECK_BUILDS_SWIFT_VERSIONS_SUCCESSFUL")
105+
builds.validateRunnerIdsSuccessful().log(check: "CHECK_BUILDS_RUNNER_IDS_SUCCESSFUL")
106106
builds.validateSuccessRateInRange().log(check: "CHECK_BUILDS_SUCCESS_RATE_IN_RANGE")
107107
}
108108
}

0 commit comments

Comments
 (0)