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.

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

Add Test-Microarchitectural to Report-Coverage dependencies #170

Merged
merged 1 commit into from
May 24, 2024

Conversation

RRozak
Copy link
Contributor

@RRozak RRozak commented May 23, 2024

The dependency to Test-Microarchitectural is missing in Report-Coverage job, but its output is required:

- name: Download coverage reports
uses: actions/download-artifact@v3
with:
name: uarch_tests_coverage_data
path: ./

I think it usually works despite missing dependency, because this job finishes sooner than other jobs listed as dependencies.

Signed-off-by: Ryszard Rozak <rrozak@antmicro.com>
@RRozak RRozak force-pushed the rrozak/coverage-dependency branch from aaeee2c to e204419 Compare May 23, 2024 14:11
Copy link

Links to coverage and verification reports for this PR (#170) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/

@tmichalak
Copy link
Collaborator

LGTM

@tmichalak tmichalak merged commit b5d5934 into chipsalliance:main May 24, 2024
165 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants