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

chore(deps): bump actions/download-artifact from 3 to 4 #190

Merged
merged 11 commits into from
Feb 8, 2024
Merged

chore(deps): bump actions/download-artifact from 3 to 4 #190

Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
1d75147
chore(deps): bump actions/download-artifact from 3 to 4
dependabot[bot] Jan 8, 2024
eb73940
Add needs: test-results to run-test.yml.
nowakweronika Jan 10, 2024
1932781
Add path "all" to download-artifact@4.
nowakweronika Jan 23, 2024
2474b1f
Delete name: "test-results" from download-artifact@4.
nowakweronika Jan 23, 2024
d3e0029
Change the oath from all to test-results.
nowakweronika Feb 7, 2024
38453d9
Fix typo.
nowakweronika Feb 7, 2024
9dfa042
Migrate artifact to v4.
nowakweronika Feb 7, 2024
6bd8501
Merge branch 'develop' into dependabot/github_actions/actions/downloa…
nowakweronika Feb 7, 2024
d42a5c7
Merge cleanup.
nowakweronika Feb 7, 2024
1fba7db
Add relevant path.
nowakweronika Feb 7, 2024
74f2ebf
Add merging results.
nowakweronika Feb 7, 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
33 changes: 22 additions & 11 deletions .github/workflows/run-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,9 @@ jobs:

- name: Upload tests results
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
name: test-results-unit
path: |
*/build/test-results/**
*/build/paparazzi/failures/**
Expand Down Expand Up @@ -144,21 +144,32 @@ jobs:

- name: Upload tests results
if: always()
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: test-results
name: test-results-instrumentation
path: |
*/build/test-results/**
retention-days: 5

merge:
runs-on: ubuntu-latest
needs:
- android-tests
- unit-tests
steps:
- name: Merge Artifacts
uses: actions/upload-artifact/merge@v4
with:
name: test-results
pattern: test-results-*

code-coverage:
name: Merged code coverage
runs-on: ubuntu-20.04
permissions:
pull-requests: write
needs:
- android-tests
- unit-tests
- merge

steps:

Expand All @@ -169,15 +180,16 @@ jobs:
uses: ./.github/actions/prepare-android-env

- name: Download tests results for both jobs
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: test-results


- name: Run code coverage
run: ./gradlew codeCoverage

- name: Store HTML coverage report
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: coverage-report
path: |
Expand Down Expand Up @@ -226,8 +238,7 @@ jobs:
runs-on: ubuntu-20.04
if: always()
needs:
- android-tests
- unit-tests
- merge
- verify-formatting
- test-license-headers

Expand All @@ -240,7 +251,7 @@ jobs:
.github/

- name: Download tests results for both jobs
uses: actions/download-artifact@v3
uses: actions/download-artifact@v4
with:
name: test-results

Expand Down
Loading