Open chlebowa opened 7 months ago
Filename Stmts Miss Cover Missing
------------------ ------- ------ ------- -------------------------------------------------------------------------------------------------------------------------------
R/adtteSpec.R 170 123 27.65% 253-401
R/assaySpec.R 54 40 25.93% 104-148
R/barplot.R 188 152 19.15% 39-66, 129-281
R/boxplot.R 190 159 16.32% 40-67, 125-280
R/checkmate.R 18 9 50.00% 110-118
R/experimentSpec.R 90 57 36.67% 97, 215-283
R/forestplot.R 214 186 13.08% 58-92, 152-328
R/geneSpec.R 257 154 40.08% 154-169, 296-482
R/km.R 208 174 16.35% 61-92, 156-327
R/pca.R 365 282 22.74% 34-56, 165-476
R/quality.R 317 244 23.03% 18-110, 208-452
R/sampleVarSpec.R 236 103 56.36% 301, 320-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-274
R/utils.R 16 0 100.00%
R/volcanoplot.R 204 174 14.71% 34-57, 113-294
R/zzz.R 2 2 0.00% 2-3
TOTAL 2710 2007 25.94%
Filename Stmts Miss Cover
----------------- ------- ------ -------
R/assaySpec.R +1 +1 -0.49%
R/barplot.R +1 +1 -0.10%
R/boxplot.R +1 +1 -0.09%
R/forestplot.R +1 +1 -0.06%
R/geneSpec.R +1 +1 -0.16%
R/km.R +1 +1 -0.08%
R/pca.R -1 -1 +0.06%
R/quality.R -2 -2 +0.14%
R/sampleVarSpec.R 0 -1 +0.42%
R/scatterplot.R +1 +1 -0.10%
R/volcanoplot.R +1 +1 -0.07%
TOTAL +5 +4 -0.01%
Results for commit: 9be50ffaeebc093e6bc64e576500b4710b96e560
Minimum allowed coverage is 80%
:recycle: This comment has been updated with latest results
1 files 15 suites 8s :stopwatch: 56 tests 43 :white_check_mark: 13 :zzz: 0 :x: 94 runs 81 :white_check_mark: 13 :zzz: 0 :x:
Results for commit 9be50ffa.
:recycle: This comment has been updated with latest results.
Thank you @danielinteractive.
Test Suite | $Status$ | Time on main |
$±Time$ | $±Tests$ | $±Skipped$ | $±Failures$ | $±Errors$ |
---|---|---|---|---|---|---|---|
adtteSpec | 💚 | $12.01$ | $-4.18$ | $-444$ | $0$ | $0$ | $0$ |
Results for commit 7c7d13eef6679453e75f03b74742db932fb0e531
♻️ This comment has been updated with latest results.
CLA Assistant Lite bot:
🎉 Thank you for your contribution! Before this PR can be accepted, we require that you all read and agree to our Contributor License Agreement.
You can digitally sign the CLA by posting a comment on this Pull Request in the format shown below. This agreement will apply to this PR as well as all future contributions on this repository.
I have read the CLA Document and I hereby sign the CLA
0 out of 3 committers have signed the CLA.
:x: @Aleksander Chlebowski
:x: @chlebowa
:x: @danielinteractive
Aleksander Chlebowski seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request
Hi @kartikeyakirar , should this be merged still or not?
Aleksander Chlebowski seems not to be a GitHub user.
Preposterous.
I have to consult with @gogonzo Looking at the changes, the update seems harmless. Is it save to merge this?
Note that we did merge similar PR in tmg:
But we didn't do it in tmc:
tmg
was merged because @gogonzo and I went through all modules to check whether they can or can not be bookmarked. Other module packages have not been rigorously tested from that angle. See here.
As discussed, we'll convert this draft for now until we're ready to surface the bookmarkable feature.
Companion to https://github.com/insightsengineering/teal/pull/1011 Replaces uses of
update*Input
with server-side input creation withrenderUI
.