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

[skip vbump] v0.1.3 release #51

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

[skip vbump] v0.1.3 release #51

wants to merge 5 commits into from

Conversation

clarkliming
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Feb 20, 2025


🎉 Thank you for your contribution! Before this PR can be accepted, we require that you 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


Liming Li (李黎明) 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. Posted by the CLA Assistant Lite bot.

Copy link
Contributor

github-actions bot commented Feb 20, 2025

badge

Code Coverage Summary

Filename          Stmts    Miss  Cover    Missing
--------------  -------  ------  -------  ------------
R/run_sas.R           6       0  100.00%
R/sas_engine.R       11       3  72.73%   5, 10, 16
R/sascfg.R            7       7  0.00%    26-32
R/utils.R            66       7  89.39%   140-145, 166
R/zzz.R               4       4  0.00%    15-18
TOTAL                94      21  77.66%

Diff against main

Filename          Stmts    Miss  Cover
--------------  -------  ------  -------
R/sas_engine.R       +2      +1  -5.05%
TOTAL                +2      +1  -0.60%

Results for commit: b2e59ec

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Feb 20, 2025

Unit Tests Summary

 1 files   5 suites   3s ⏱️
16 tests 12 ✅ 4 💤 0 ❌
21 runs  16 ✅ 5 💤 0 ❌

Results for commit 5d81bae.

♻️ This comment has been updated with latest results.

Copy link
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
sascfg 💔 $0.13$ $+3.21$ $0$ $-1$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
sascfg 💔 $0.13$ $+3.21$ sascfg_creates_sas_configuration_file

Results for commit 0a0d678

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant