Remove `configure_ci.py` from `post_benchmark_comment.yaml` (#14343)
Jobs in `post_benchmark_comment.yaml` has no dependency on the outputs
of `configure_ci.py`. Remove the configure job.
skip-ci: Not run in presubmit
diff --git a/.github/workflows/post_benchmark_comment.yaml b/.github/workflows/post_benchmark_comment.yaml
index 66abd0f..8b029f5 100644
--- a/.github/workflows/post_benchmark_comment.yaml
+++ b/.github/workflows/post_benchmark_comment.yaml
@@ -17,34 +17,10 @@
PR_CI_RUN_ATTEMPT: ${{ github.event.workflow_run.run_attempt }}
jobs:
- setup:
+ post_comment:
# Only run the workflow if it's triggered from a pull request.
if: github.event.workflow_run.event == 'pull_request'
runs-on: ubuntu-20.04
- outputs:
- should-run: ${{ steps.configure.outputs.should-run }}
- runner-env: ${{ steps.configure.outputs.runner-env }}
- runner-group: ${{ steps.configure.outputs.runner-group }}
- steps:
- - name: "Checking out repository"
- uses: actions/checkout@8f4b7f84864484a7bf31766abe9204da3cbe65b3 # v3.5.0
- with:
- # We need the parent commit to do a diff
- fetch-depth: 2
- - name: "Configuring CI options"
- id: configure
- run: |
- # Just informative logging. There should only be two commits in the
- # history here, but limiting the depth helps when copying from a local
- # repo instead of using checkout, e.g. with
- # https://github.com/nektos/act where there will be more.
- git log --oneline --graph --max-count=3
- ./build_tools/github_actions/configure_ci.py
-
- post_comment:
- needs: setup
- if: needs.setup.outputs.should-run == 'true'
- runs-on: ubuntu-20.04
permissions:
issues: write
pull-requests: write