Skip to content

Commit 5ba60c4

Browse files
authored
chore: Remove integration tests on fork approval (#224)
1 parent 0f058a2 commit 5ba60c4

File tree

1 file changed

+2
-50
lines changed

1 file changed

+2
-50
lines changed

.github/workflows/python_integration_tests.yaml

Lines changed: 2 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,12 @@ concurrency:
1919

2020
jobs:
2121
# Job to run integration tests from the main repository.
22-
integration_tests_on_main:
22+
integration_tests:
2323
name: Integration tests
2424
runs-on: ubuntu-latest
2525
# Run this only for PRs from the main repository or for pushes to master. Skip otherwise.
2626
if: github.event.pull_request.head.repo.owner.login == 'apify' || github.ref == 'refs/heads/master'
27+
2728
strategy:
2829
matrix:
2930
python-version: ${{ fromJSON(inputs.python-versions)}}
@@ -50,52 +51,3 @@ jobs:
5051
run: make INTEGRATION_TESTS_CONCURRENCY=8 integration-tests
5152
env:
5253
APIFY_TEST_USER_API_TOKEN: ${{ secrets.APIFY_TEST_USER_PYTHON_SDK_API_TOKEN }}
53-
54-
# Job to request approval before running integration tests for PRs from forks.
55-
integration_tests_on_fork_approval:
56-
name: Approve integration tests for PRs from forks
57-
runs-on: ubuntu-latest
58-
# Run this only for PRs from forks. Skip otherwise.
59-
if: github.event.pull_request.head.repo.owner.login != 'apify' && github.ref != 'refs/heads/master'
60-
environment:
61-
name: fork-pr-integration-tests
62-
url: ${{ github.event.pull_request.html_url }}
63-
steps:
64-
- name: Await approval
65-
run: echo "Waiting for approval to run integration tests for fork PR."
66-
67-
# Job to run integration tests for PRs from forks, only after manual approval.
68-
integration_tests_on_fork:
69-
name: Integration tests for PRs from forks
70-
needs: integration_tests_on_fork_approval
71-
# Run this only for PRs from forks. Skip otherwise.
72-
if: github.event.pull_request.head.repo.owner.login != 'apify' && github.ref != 'refs/heads/master'
73-
runs-on: ubuntu-latest
74-
strategy:
75-
matrix:
76-
python-version: ${{ fromJSON(inputs.python-versions)}}
77-
max-parallel: 1 # No parallel tests to avoid overshooting limits.
78-
79-
steps:
80-
- name: Checkout repository
81-
uses: actions/checkout@v4
82-
83-
- name: Set up Python ${{ matrix.python-version }}
84-
uses: actions/setup-python@v5
85-
with:
86-
python-version: ${{ matrix.python-version }}
87-
88-
- name: Set up Python ${{ matrix.python-version }}
89-
uses: actions/setup-python@v5
90-
with:
91-
python-version: ${{ matrix.python-version }}
92-
93-
- name: Set up uv package manager
94-
uses: astral-sh/setup-uv@v6
95-
with:
96-
python-version: ${{ matrix.python-version }}
97-
98-
- name: Run integration tests
99-
run: make INTEGRATION_TESTS_CONCURRENCY=8 integration-tests
100-
env:
101-
APIFY_TEST_USER_API_TOKEN: ${{ secrets.APIFY_TEST_USER_PYTHON_SDK_API_TOKEN }}

0 commit comments

Comments
 (0)