Skip to content

rerun docs and update news#287

Merged
donyunardi merged 1 commit into
mainfrom
run-docs@main
Apr 23, 2026
Merged

rerun docs and update news#287
donyunardi merged 1 commit into
mainfrom
run-docs@main

Conversation

@donyunardi
Copy link
Copy Markdown
Contributor

I was too fast when approving #286

In this PR, I'm updating docs and add records in news.
Not sure why CI/CD passing R CMD check (which need to look into it).

Copy link
Copy Markdown
Contributor

@osenan osenan left a comment

Choose a reason for hiding this comment

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

LGTM!
For the investigation on why the CI did not fail, running devtools::check() on main before this PR does not fail locally. This suggests that there is nothing wrong in the CI configuration, but the check itself does not fail in such small difference? It might be that the check does not fail for a single comma? But we should confirm it.

@github-actions
Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   14 suites   7s ⏱️
187 tests 183 ✅ 4 💤 0 ❌
270 runs  266 ✅ 4 💤 0 ❌

Results for commit 70148a3.

@github-actions
Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   14 suites   6s ⏱️
187 tests 183 ✅ 4 💤 0 ❌
270 runs  266 ✅ 4 💤 0 ❌

Results for commit 70148a3.

@github-actions
Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                         Stmts    Miss  Cover    Missing
-----------------------------  -------  ------  -------  -------------
R/qenv-c.R                          55       0  100.00%
R/qenv-class.R                      13       0  100.00%
R/qenv-concat.R                      7       0  100.00%
R/qenv-constructor.R                 1       0  100.00%
R/qenv-errors.R                      4       4  0.00%    6-9
R/qenv-eval_code.R                  63       1  98.41%   40
R/qenv-extract.R                    30       0  100.00%
R/qenv-get_code.R                   24       0  100.00%
R/qenv-get_env.R                     3       1  66.67%   27
R/qenv-get_messages.r                5       0  100.00%
R/qenv-get_outputs.R                 6       0  100.00%
R/qenv-get_var.R                    13       1  92.31%   13
R/qenv-get_warnings.R                5       0  100.00%
R/qenv-join.R                        1       1  0.00%    13
R/qenv-length.R                      2       1  50.00%   2
R/qenv-show.R                       29      29  0.00%    19-50
R/qenv-within.R                      8       0  100.00%
R/utils-get_code_dependency.R      266       3  98.87%   160, 258, 341
R/utils.R                           42       0  100.00%
TOTAL                              577      41  92.89%

Diff against main

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

Results for commit: 70148a3

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@donyunardi donyunardi merged commit 7e3d09e into main Apr 23, 2026
29 checks passed
@donyunardi donyunardi deleted the run-docs@main branch April 23, 2026 19:25
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants