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

rm staged dep #170

Merged
merged 2 commits into from
Oct 29, 2024
Merged

rm staged dep #170

merged 2 commits into from
Oct 29, 2024

Conversation

shajoezhu
Copy link
Contributor

close #169

Copy link
Contributor

github-actions bot commented Oct 25, 2024

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@shajoezhu
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Copy link
Contributor

github-actions bot commented Oct 25, 2024

badge

Code Coverage Summary

Filename               Stmts    Miss  Cover    Missing
-------------------  -------  ------  -------  ---------
R/assertions.R            73       0  100.00%
R/co_relevels.R           17       0  100.00%
R/cut_by_group.R          15       0  100.00%
R/explicit_na.R           26       0  100.00%
R/filter.R                75       0  100.00%
R/join_adsub_adsl.R       92       8  91.30%   166-173
R/pivot.R                 59       0  100.00%
R/propagate.R             22       0  100.00%
R/reformat.R              80       0  100.00%
R/render_safe.R           55       1  98.18%   50
R/rules.R                 72       0  100.00%
R/unite.R                 31       0  100.00%
R/utils.R                 21       2  90.48%   72-73
R/zzz.R                    3       3  0.00%    4-6
TOTAL                    641      14  97.82%

Diff against main

Filename               Stmts    Miss  Cover
-------------------  -------  ------  --------
R/assertions.R           +19       0  +100.00%
R/filter.R                +2       0  +100.00%
R/join_adsub_adsl.R      +26       0  +3.43%
R/pivot.R                -14       0  +100.00%
R/reformat.R             +25       0  +100.00%
R/render_safe.R          +17       0  +0.81%
R/rules.R                +19      -1  +1.89%
R/utils.R                 -6      +2  -9.52%
TOTAL                    +88      +1  +0.17%

Results for commit: 61be08e

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Oct 25, 2024

Unit Tests Summary

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

Results for commit 61be08e.

♻️ This comment has been updated with latest results.

@shajoezhu shajoezhu merged commit 7436845 into main Oct 29, 2024
26 checks passed
@shajoezhu shajoezhu deleted the 169_rm_stageddep branch October 29, 2024 08:26
@github-actions github-actions bot locked and limited conversation to collaborators Oct 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

dunlin switch from staged-dependencies to setup-r-dependencies for deps install
2 participants