diff --git a/.github/workflows/lintdiff-code.yaml b/.github/workflows/lintdiff-code.yaml index 8d3110e50b90..fa7ec6f0f80f 100644 --- a/.github/workflows/lintdiff-code.yaml +++ b/.github/workflows/lintdiff-code.yaml @@ -1,4 +1,4 @@ -name: "[TEST-IGNORE] Swagger LintDiff - Analyze Code" +name: "Swagger LintDiff - Analyze Code" on: pull_request @@ -7,7 +7,7 @@ permissions: jobs: lintdiff: - name: "[TEST-IGNORE] Swagger LintDiff - Analyze Code" + name: "Swagger LintDiff - Analyze Code" runs-on: ubuntu-24.04 steps: @@ -49,12 +49,6 @@ jobs: writeFileSync(filePath, changedFiles.join('\n'), 'utf8'); console.log(`Changed files written to ${filePath}`); - - name: Prepend test notice to summary - if: always() - run: | - temp_file=$(mktemp) - echo -e "> [!IMPORTANT]\n> This check is testing a new version of 'Swagger LintDiff'.\n> Failures are expected, and should be completely ignored by spec authors and reviewers.\n> Meaningful results for this PR are are in required check 'Swagger LintDiff'.\n" > $GITHUB_STEP_SUMMARY - # TODO: Could be github.sha for merge commit - name: Run LintDiff run: | @@ -65,10 +59,6 @@ jobs: --base-branch ${{ github.event.pull_request.base.ref }} \ --compare-sha ${{ github.event.pull_request.head.sha }} \ --out-file $GITHUB_STEP_SUMMARY - - echo "⚠️ This check is testing a new version of 'Swagger LintDiff'." - echo "⚠️ Failures are expected, and should be completely ignored by spec authors and reviewers." - echo "⚠️ Meaningful results for this PR are are in required check 'Swagger LintDiff'." env: # Some LintDiff runs are memory intensive and require more than the # default. diff --git a/.github/workflows/lintdiff-status.yaml b/.github/workflows/lintdiff-status.yaml index a7f651a7641a..da90f5348c9d 100644 --- a/.github/workflows/lintdiff-status.yaml +++ b/.github/workflows/lintdiff-status.yaml @@ -1,4 +1,4 @@ -name: "[TEST-IGNORE] Swagger LintDiff - Set Status" +name: "Swagger LintDiff - Set Status" on: # Must run on pull_request_target instead of pull_request, since the latter cannot trigger on @@ -15,7 +15,7 @@ on: - labeled - unlabeled workflow_run: - workflows: ["\\[TEST-IGNORE\\] Swagger LintDiff - Analyze Code"] + workflows: ["Swagger LintDiff - Analyze Code"] types: [completed] permissions: @@ -30,6 +30,6 @@ jobs: name: Set LintDiff Status uses: ./.github/workflows/_reusable-set-check-status.yml with: - monitored_workflow_name: "[TEST-IGNORE] Swagger LintDiff - Analyze Code" - required_check_name: "[TEST-IGNORE] Swagger LintDiff" + monitored_workflow_name: "Swagger LintDiff - Analyze Code" + required_check_name: "Swagger LintDiff" overriding_label: "Approved-LintDiff" diff --git a/.github/workflows/watch-lintdiff.yaml b/.github/workflows/watch-lintdiff.yaml deleted file mode 100644 index 7eb843df3f4d..000000000000 --- a/.github/workflows/watch-lintdiff.yaml +++ /dev/null @@ -1,25 +0,0 @@ -# Use ~ to sort the workflow to the bottom of the list -name: "~Watch - LintDiff" - -on: - # check_suite is preferred over check_run to avoid triggering on all check - # runs. In some cases, check_run must be used in testing environments. - check_suite: - types: completed - - workflow_run: - types: completed - workflows: - - "\\[TEST-IGNORE\\] Swagger LintDiff - Analyze Code" - -permissions: - checks: read - contents: read - -jobs: - LintDiff: - name: Watch LintDiff - uses: ./.github/workflows/_reusable-verify-run-status.yaml - with: - check_run_name: "Swagger LintDiff" - workflow_name: "[TEST-IGNORE] Swagger LintDiff - Analyze Code"