Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GHA trigger CI with pull_request_target for secrets access #1132

Merged
merged 3 commits into from
Nov 13, 2024
+24 −34
Merged

GHA trigger CI with pull_request_target for secrets access #1132

Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
262362f
Update CI workflow to trigger with pull_request_target
AndreKurait Nov 13, 2024
04b4482
Remove duplicate codecov testing
AndreKurait Nov 13, 2024
75fafba
Sort codecov parameters in github actions
AndreKurait Nov 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
55 changes: 23 additions & 32 deletions .github/workflows/CI.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: CI

on:
push:
pull_request:
pull_request_target:

env:
python-version: '3.11'
Expand Down Expand Up @@ -63,17 +63,14 @@ jobs:
pipenv install --deploy --dev
pipenv run test
pipenv run coverage xml
- name: Upload Coverage with retry
uses: Wandalen/wretry.action@v3.7.2
- name: Upload Coverage
uses: codecov/codecov-action@v4
with:
attempt_limit: 5
attempt_delay: 2000
action: codecov/codecov-action@v4
with: |
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
flags: python-test
fail_ci_if_error: true
verbose: true
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
flags: python-test
fail_ci_if_error: true

console-api-tests:
runs-on: ubuntu-latest
Expand All @@ -90,17 +87,14 @@ jobs:
pipenv install --deploy --dev
pipenv run coverage run --source='.' manage.py test console_api
pipenv run coverage xml
- name: Upload Coverage with retry
uses: Wandalen/wretry.action@v3.7.2
- name: Upload Coverage
uses: codecov/codecov-action@v4
with:
attempt_limit: 5
attempt_delay: 2000
action: codecov/codecov-action@v4
with: |
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
flags: python-test
fail_ci_if_error: true
verbose: true
token: ${{ secrets.CODECOV_TOKEN }}
files: ./coverage.xml
flags: python-test
fail_ci_if_error: true


gradle-tests:
Expand Down Expand Up @@ -132,18 +126,15 @@ jobs:
path: |
**/build/reports/tests/
**/reports/jacoco/mergedReport/
- name: Upload Coverage with retry
uses: Wandalen/wretry.action@v3.7.2
- name: Upload Coverage
uses: codecov/codecov-action@v4
with:
attempt_limit: 5
attempt_delay: 2000
action: codecov/codecov-action@v4
with: |
token: ${{ secrets.CODECOV_TOKEN }}
files: ${{ github.workspace }}/build/reports/jacoco/mergedReport/jacocoMergedReport.xml
flags: gradle-test
fail_ci_if_error: true
disable_search: true
verbose: true
token: ${{ secrets.CODECOV_TOKEN }}
fail_ci_if_error: true
AndreKurait marked this conversation as resolved.
Show resolved Hide resolved
files: ${{ github.workspace }}/build/reports/jacoco/mergedReport/jacocoMergedReport.xml
flags: gradle-test
disable_search: true



Expand Down