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

Remove "if: always()" from tests.yml and bump all pydpf-actions to v2.3 #1685

Merged
merged 7 commits into from
Aug 9, 2024

Conversation

PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Aug 5, 2024

Also switch to artifact-upload@v4 and fix concurrent artifact upload

@PProfizi PProfizi added the CI/CD Related to CI/CD label Aug 5, 2024
@PProfizi PProfizi added this to the v0.13.1 milestone Aug 5, 2024
@PProfizi PProfizi self-assigned this Aug 5, 2024
….3 and fix concurrent artifact-upload

Signed-off-by: paul.profizi <[email protected]>
Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.09%. Comparing base (da757b4) to head (ea984f3).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1685      +/-   ##
==========================================
- Coverage   82.04%   81.09%   -0.96%     
==========================================
  Files          83       83              
  Lines        9937     9937              
==========================================
- Hits         8153     8058      -95     
- Misses       1784     1879      +95     

@PProfizi PProfizi requested a review from BClappe August 5, 2024 16:43
.github/workflows/tests.yml Outdated Show resolved Hide resolved
Co-authored-by: BrunoClappe-Ansys <[email protected]>
@PProfizi PProfizi merged commit b7cc169 into master Aug 9, 2024
43 checks passed
@PProfizi PProfizi deleted the ci/remove_if_always branch August 9, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Related to CI/CD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants