[webkit-changes] [WebKit/WebKit] 2ed7ca: [Static Analyzer CI] Safer CPP build should skip P...

[Brianna Fan]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2ed7ca28c53b094403638a92ba3afa01e97b8bc2
      https://github.com/WebKit/WebKit/commit/2ed7ca28c53b094403638a92ba3afa01e97b8bc2
  Author: Brianna Fan <bfan2 at apple.com>
  Date:   2024-10-08 (Tue, 08 Oct 2024)

  Changed paths:
    M Tools/CISupport/ews-build/factories.py
    M Tools/CISupport/ews-build/factories_unittest.py
    M Tools/CISupport/ews-build/steps.py
    M Tools/CISupport/ews-build/steps_unittest.py

  Log Message:
  -----------
  [Static Analyzer CI] Safer CPP build should skip PRs without relevant changes or with skip-ews label
https://bugs.webkit.org/show_bug.cgi?id=281010
rdar://137451787

Reviewed by Ryan Haddad.

Add ValidateChanges after checking out PR to check if skip-ews label is applied.
Check whether changes in the PR are under Source/WebKit or Source/WebCore or are in any of the
scripts associated with the Safer CPP queue.

This will skip unnecessary builds and allow the queue to process more relevant changes in a timely fashion.

* Tools/CISupport/ews-build/factories.py:
(SaferCPPStaticAnalyzerFactory.__init__):
* Tools/CISupport/ews-build/factories_unittest.py:
(TestExpectedBuildSteps):
* Tools/CISupport/ews-build/steps.py:
(CheckChangeRelevance):
* Tools/CISupport/ews-build/steps_unittest.py:
(TestCheckChangeRelevance.test_relevant_safer_cpp_pull_request):
(TestCheckChangeRelevance.test_non_relevant_patch_on_various_queues):
(TestCheckChangeRelevance.test_non_relevant_pull_request_on_various_queues):

Canonical link: https://commits.webkit.org/284833@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications