[GitHub] Remove pr-subscriber-wait.py download

[DavidSpickett]

This file and uses of it were removed in 64d5da6.

[llvmbot]

@llvm/pr-subscribers-github-workflow

Author: David Spickett (DavidSpickett)

Changes

Uses of this file were removed in 64d5da6.

---

Full diff: https://github.com/llvm/llvm-project/pull/72120.diff

1 Files Affected:

• (modified) .github/workflows/pr-subscriber.yml (-1)

diff --git a/.github/workflows/pr-subscriber.yml b/.github/workflows/pr-subscriber.yml
index ef2ef7b9e4a35c1..03f60f619568f11 100644
--- a/.github/workflows/pr-subscriber.yml
+++ b/.github/workflows/pr-subscriber.yml
@@ -17,7 +17,6 @@ jobs:
         run: |
           curl -O -L https://raw.githubusercontent.com/"$GITHUB_REPOSITORY"/main/llvm/utils/git/github-automation.py
           curl -O -L https://raw.githubusercontent.com/"$GITHUB_REPOSITORY"/main/llvm/utils/git/requirements.txt
-          curl -O -L https://raw.githubusercontent.com/"$GITHUB_REPOSITORY"/main/.github/workflows/pr-subscriber-wait.py
           chmod a+x github-automation.py
           pip install -r requirements.txt
 

[tstellar]

Is the file still there too?

[DavidSpickett]

No it was removed also, I've clarified the PR description.

[boomanaiden154]

After seeing this patch, I was wondering why the job didn't just outright fail. Apparently curl doesn't return a non-zero exit code for HTTP requests that return a code other than 200. Thanks for the cleanup!

[DavidSpickett]

Apparently curl doesn't return a non-zero exit code for HTTP requests that return a code other than 200.

If you happen to know the option to make curl fail earlier that would be a good change to make.

[tru]

--fail is the right one I think.

[DavidSpickett]

If you happen to know the option to make curl fail earlier that would be a good change to make.

#72238