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 setup-r-dependencies #395

Merged
merged 4 commits into from
Dec 12, 2024
Merged

add setup-r-dependencies #395

merged 4 commits into from
Dec 12, 2024

Conversation

vedhav
Copy link
Contributor

@vedhav vedhav commented Dec 10, 2024

Related to insightsengineering/nestdevs-tasks#65
Switch to setup-r-dependencies.
Added teal.modules.hermes for lookup-refs.

@vedhav vedhav force-pushed the setup-r-dependencies@main branch from 9f08183 to ae1976f Compare December 10, 2024 11:39
Copy link
Contributor

github-actions bot commented Dec 10, 2024

badge

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  -------------------------------------------------------------------------------------------------------------------------------
R/adtteSpec.R           171     124  27.49%   256-404
R/assaySpec.R            54      40  25.93%   104-147
R/barplot.R             188     152  19.15%   39-66, 129-280
R/boxplot.R             190     159  16.32%   40-67, 125-279
R/checkmate.R            18       9  50.00%   110-118
R/experimentSpec.R       91      58  36.26%   97, 215-284
R/forestplot.R          214     186  13.08%   60-94, 154-329
R/geneSpec.R            257     154  40.08%   154-169, 296-481
R/km.R                  208     174  16.35%   63-94, 158-328
R/pca.R                 367     284  22.62%   34-56, 165-477
R/quality.R             320     247  22.81%   18-110, 208-454
R/sampleVarSpec.R       237     104  56.12%   300, 319-328, 334-341, 343, 351-363, 365-366, 368, 371, 379-383, 385-400, 405-429, 432-436, 438, 445-455, 457-458, 466, 511-528
R/scatterplot.R         181     148  18.23%   39-65, 127-273
R/utils.R                16       0  100.00%
R/volcanoplot.R         204     174  14.71%   34-57, 113-293
R/zzz.R                   2       2  0.00%    2-3
TOTAL                  2718    2015  25.86%

Diff against main

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

Results for commit: 160cd5c

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Dec 10, 2024

Unit Tests Summary

 1 files  15 suites   8s ⏱️
56 tests 43 ✅ 13 💤 0 ❌
94 runs  81 ✅ 13 💤 0 ❌

Results for commit 160cd5c.

♻️ This comment has been updated with latest results.

@m7pr
Copy link
Contributor

m7pr commented Dec 10, 2024

@vedhav R CMD CHECK is failing in here

@m7pr m7pr self-assigned this Dec 10, 2024
Copy link
Contributor

@m7pr m7pr left a comment

Choose a reason for hiding this comment

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

besides failiing r cmd check this is good to go

@vedhav vedhav enabled auto-merge (squash) December 12, 2024 14:15
@vedhav vedhav merged commit d059ae3 into main Dec 12, 2024
30 checks passed
@vedhav vedhav deleted the setup-r-dependencies@main branch December 12, 2024 14:33
@github-actions github-actions bot locked and limited conversation to collaborators Dec 12, 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.

2 participants