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

Fix broken pipelines #280

Merged
merged 24 commits into from
Oct 8, 2024
Merged

Fix broken pipelines #280

merged 24 commits into from
Oct 8, 2024

Conversation

m7pr
Copy link
Contributor

@m7pr m7pr commented Aug 28, 2024

Fix #279

@m7pr m7pr added the core label Aug 28, 2024
@m7pr m7pr requested a review from donyunardi August 30, 2024 10:41
@m7pr m7pr marked this pull request as ready for review October 3, 2024 06:56
@m7pr
Copy link
Contributor Author

m7pr commented Oct 3, 2024

Since we provided a fix in insightsengineering/r-revdepcheck-action#4 for the last build, I am opening this PR and restarting the build

Copy link
Contributor

github-actions bot commented Oct 3, 2024

badge

Code Coverage Summary

Filename                    Stmts    Miss  Cover    Missing
------------------------  -------  ------  -------  --------------
R/tm_g_ae_oview.R             246     246  0.00%    82-360
R/tm_g_ae_sub.R               309     309  0.00%    59-403
R/tm_g_butterfly.R            382     382  0.00%    123-544
R/tm_g_decorate.R              47      47  0.00%    18-99
R/tm_g_events_term_id.R       293     293  0.00%    62-392
R/tm_g_heat_bygrade.R         316     316  0.00%    134-485
R/tm_g_patient_profile.R      712     712  0.00%    156-935
R/tm_g_spiderplot.R           330     330  0.00%    98-474
R/tm_g_swimlane.R             370     370  0.00%    125-551
R/tm_g_waterfall.R            440     440  0.00%    107-602
R/utils.R                      47      42  10.64%   32-91, 124-160
R/zzz.R                         2       2  0.00%    2-3
TOTAL                        3494    3489  0.14%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 5a2c8c6

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Oct 3, 2024

Unit Tests Summary

 1 files   1 suites   0s ⏱️
 4 tests  4 ✅ 0 💤 0 ❌
13 runs  13 ✅ 0 💤 0 ❌

Results for commit 5a2c8c6.

♻️ This comment has been updated with latest results.

@m7pr
Copy link
Contributor Author

m7pr commented Oct 3, 2024

Checks are failiing as there is external cairo dependency missing. Raised this to IDR for help

The same as for teal.goshawk insightsengineering/teal.goshawk#306 (comment)

R/tm_g_ae_oview.R Outdated Show resolved Hide resolved
DESCRIPTION Show resolved Hide resolved
@m7pr
Copy link
Contributor Author

m7pr commented Oct 8, 2024

NEWS.md Outdated Show resolved Hide resolved
Co-authored-by: Pawel Rucki <[email protected]>
Signed-off-by: Marcin <[email protected]>
NEWS.md Outdated Show resolved Hide resolved
Signed-off-by: Pawel Rucki <[email protected]>
NEWS.md Outdated Show resolved Hide resolved
Signed-off-by: Pawel Rucki <[email protected]>
Copy link
Contributor

@pawelru pawelru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@pawelru pawelru self-assigned this Oct 8, 2024
@m7pr m7pr enabled auto-merge (squash) October 8, 2024 13:48
@m7pr m7pr merged commit 44e1147 into main Oct 8, 2024
30 checks passed
@m7pr m7pr deleted the fix-pipelines@main branch October 8, 2024 13:48
@github-actions github-actions bot locked and limited conversation to collaborators Oct 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix failed pipeline
3 participants