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

ci: Revert breaking change to publish-unit-test-result-action #2942

Conversation

rbarker-dev
Copy link
Contributor

Description:

Previously the publish-unit-test-result action was updated to use the step-security maintained version of publish-unit-test-result-action. However, the step-security action is a fork of the enricoCI action. This change broke the actionite publish steps.

Related Issue(s):

Fixes: #2941

**Description**:

Previously the publish-unit-test-result action was updated to use the
step-security maintained version of publish-unit-test-result-action.
However, the step-security action is a fork of the enricoCI action.
This change broke the actionite publish steps.

**Related Issue(s)**:

Fixes: #2941

Signed-off-by: Roger Barker <[email protected]>
@rbarker-dev rbarker-dev added bug Something isn't working github_action labels Sep 5, 2024
@rbarker-dev rbarker-dev requested a review from san-est September 5, 2024 14:08
@rbarker-dev rbarker-dev self-assigned this Sep 5, 2024
@rbarker-dev rbarker-dev linked an issue Sep 5, 2024 that may be closed by this pull request
Copy link

sonarqubecloud bot commented Sep 5, 2024

@rbarker-dev rbarker-dev added this to the 0.56.0 milestone Sep 5, 2024
Copy link

github-actions bot commented Sep 5, 2024

Acceptance Tests

  20 files  ±    0  293 suites  +15   33m 6s ⏱️ +57s
602 tests ±    0  593 ✔️ +    2  4 💤 ±0  5  - 2 
923 runs  +133  910 ✔️ +133  4 💤 ±0  9 ±0 

Results for commit cc7c7ea. ± Comparison against base commit f992ce1.

@rbarker-dev rbarker-dev merged commit 6ecdd7c into main Sep 5, 2024
43 of 47 checks passed
@rbarker-dev rbarker-dev deleted the 2941-publish-unit-test-result-action-update-breaks-ci-checks branch September 5, 2024 14:34
Copy link

codecov bot commented Sep 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.20%. Comparing base (f992ce1) to head (cc7c7ea).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2942      +/-   ##
==========================================
+ Coverage   84.17%   84.20%   +0.03%     
==========================================
  Files          56       40      -16     
  Lines        3753     3039     -714     
  Branches      773      634     -139     
==========================================
- Hits         3159     2559     -600     
+ Misses        355      278      -77     
+ Partials      239      202      -37     
Flag Coverage Δ
relay 84.20% <ø> (ø)
server ?
ws-server ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 16 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working github_action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

publish-unit-test-result-action update breaks CI checks
3 participants