ci: removed event_name == 'push' actions

[sirandreww-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[sirandreww-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• ci: removed event_name == 'push' actions #10230 👈 (View in Graphite)
• ci: run all the integration tests on PR or push #10084
• main-v0.14.1

This stack of pull requests is managed by Graphite. Learn more about stacking.

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @Itay-Tsabary-Starkware and @ShahakShama)

---

.github/workflows/main.yml line 9 at r1 (raw file):

      - main-v[0-9].**
    tags:
      - v[0-9].**

you can delete this, I think

Code quote:

  push:
    branches:
      - main
      - main-v[0-9].**
    tags:
      - v[0-9].**

---

.github/workflows/main.yml line 80 at r1 (raw file):

      - name: "Run python code style pull request"
        if: github.event_name == 'pull_request'
        run: scripts/py_code_style.py

see above - pull_request this is the only event

Suggestion:

      # Run code style on PR.
      - name: "Run TODO style"
        run: scripts/named_todos.py --commit_id ${{ github.event.pull_request.base.sha }}
      - name: "Run clippy"
        run: scripts/run_tests.py --command clippy --changes_only --commit_id ${{ github.event.pull_request.base.sha }}
      - name: "Run cargo doc"
        run: scripts/run_tests.py --command doc --changes_only --commit_id ${{ github.event.pull_request.base.sha }}
      - name: "Run python code style"
        run: scripts/py_code_style.py

---

.github/workflows/main.yml line 128 at r1 (raw file):

        if: github.event_name == 'pull_request'
        run: |
          scripts/run_tests.py --command test --changes_only --include_dependencies --commit_id ${{ github.event.pull_request.base.sha }}

see above

Suggestion:

      - name: "Run tests pull request"
        run: |
          scripts/run_tests.py --command test --changes_only --include_dependencies --commit_id ${{ github.event.pull_request.base.sha }}

---

.github/workflows/main.yml line 134 at r1 (raw file):

  integration-tests-pr:
    if: github.event_name == 'pull_request'
    uses: ./.github/workflows/integration-tests.yml

see above

Suggestion:

  integration-tests-pr:
    uses: ./.github/workflows/integration-tests.yml

[sirandreww-starkware]

Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @dorimedini-starkware, @Itay-Tsabary-Starkware, and @ShahakShama)

---

.github/workflows/main.yml line 9 at r1 (raw file):

Previously, dorimedini-starkware wrote…

you can delete this, I think

Done.

---

.github/workflows/main.yml line 80 at r1 (raw file):

Previously, dorimedini-starkware wrote…

see above - pull_request this is the only event

Done.

---

.github/workflows/main.yml line 128 at r1 (raw file):

Previously, dorimedini-starkware wrote…

see above

Done.

---

.github/workflows/main.yml line 134 at r1 (raw file):

Previously, dorimedini-starkware wrote…

see above

Done.

[Itay-Tsabary-Starkware]

Conceptual

Reviewable status: 0 of 1 files reviewed, 4 unresolved discussions (waiting on @dorimedini-starkware and @ShahakShama)

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ShahakShama)

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ShahakShama)

[dorimedini-starkware]

@dorimedini-starkware reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @ShahakShama)