Closed cicdguy closed 2 years ago
Filename Stmts Miss Cover Missing
----------------------- ------- ------ ------- -------------------------------------------------------------------------------------------------------
R/caching.R 143 3 97.90% 68, 127-128
R/dependencies_app.R 117 117 0.00% 35-182
R/dependencies_helper.R 221 64 71.04% 40-41, 45, 50-53, 86-102, 108-116, 121-128, 134-135, 140-145, 152, 161-166, 220, 254, 266-282, 293, 403
R/dependencies.R 308 38 87.66% 78, 85, 102, 119, 194-208, 250, 258-261, 263, 275, 359, 421, 462-473, 520, 588-589, 596, 683
R/git_tools.R 160 73 54.37% 55, 71, 89-96, 99, 103, 120, 129-130, 146, 149-156, 169-180, 220-291
R/graph_methods.R 69 1 98.55% 114
R/host.R 19 10 47.37% 6-9, 20-26
R/ref_strategy.R 40 0 100.00%
R/rstudio_addins.R 5 5 0.00% 6-22
R/rstudio_jobs.R 27 27 0.00% 3-70
R/utils.R 92 6 93.48% 2, 8, 72-73, 82, 92
R/zzz.R 18 10 44.44% 27-37
TOTAL 1219 354 70.96%
Results for commit: 311599d956b7c3259ae48926ac720f7b7478218e
Minimum allowed coverage is 80%
:recycle: This comment has been updated with latest results
1 files 12 suites 13s :stopwatch: 62 tests 62 :heavy_check_mark: 0 :zzz: 0 :x: 195 runs 195 :heavy_check_mark: 0 :zzz: 0 :x:
Results for commit ea05beb7.
:recycle: This comment has been updated with latest results.
Also:
staged.deps uses renv so fixed versions of packages - over time these will stray further and further from what's in the docker container adding time to reinstall all the packages? Just worth keeping in mind for if other packages start using renv (i.e. to make them consistent)
I assume it's ok using staged.deps itself in the creating environment workflow - though as there is no staged.deps yaml file here only renv I would have expected a nicer name of the action and it not to be referenced at all
The underlying action is actually multi-faceted in that it installs from renv when there is a lock file available: https://github.com/insightsengineering/staged-dependencies-action/blob/main/staged-dependencies.R#L55-L56
Shared workflows = lesser maintenance.