Skip to content

Small changes in documentation after CI changes#71

Closed
osenan wants to merge 1 commit intomainfrom
fix-documentation-iteraction@main
Closed

Small changes in documentation after CI changes#71
osenan wants to merge 1 commit intomainfrom
fix-documentation-iteraction@main

Conversation

@osenan
Copy link
Copy Markdown
Contributor

@osenan osenan commented May 6, 2026

After update on roxygen2 package on CI, there are small changes that need to done if not the roxygen action on documentation will have errors.

This PR updates the DESCRIPTION Roxygen2 configuration (already happening on some branches) and makes a change so devtools::document() can be called with the new version of devtools.

@osenan osenan requested a review from a team May 6, 2026 12:19
@osenan osenan added the core label May 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

badge

Code Coverage Summary

Filename                  Stmts    Miss  Cover    Missing
----------------------  -------  ------  -------  -------------------------------------------------------------
R/as_picks.R                140      21  85.00%   63, 167-171, 182-186, 201-207, 229-231
R/assertion.R                 5       0  100.00%
R/call_utils.R              147      33  77.55%   23-28, 65, 132-138, 259, 278-289, 292, 317-325
R/helpers.R                  12      12  0.00%    4-15
R/interaction.R              42      26  38.10%   39-97
R/module_merge.R            266      14  94.74%   303, 310-320, 337, 616
R/module_picks.R            318      23  92.77%   47-53, 71, 109-110, 298-300, 302-306, 429, 478, 516, 528, 536
R/picks.R                   172       1  99.42%   335
R/print.R                    36       2  94.44%   50, 58
R/resolver.R                141      19  86.52%   80-83, 110-118, 284-289
R/tidyselect-helpers.R       29       0  100.00%
R/tm_merge.R                 54      54  0.00%    42-102
R/ui_containers.R            42       0  100.00%
R/zzz.R                       5       5  0.00%    3-11
TOTAL                      1409     210  85.10%

Diff against main

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

Results for commit: f584ee3

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Unit Tests Summary

  1 files   10 suites   18s ⏱️
255 tests 241 ✅ 14 💤 0 ❌
383 runs  368 ✅ 15 💤 0 ❌

Results for commit f584ee3.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
badge_dropdown 💀 $0.16$ $-0.16$ $-1$ $-1$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
assertion 👶 $+0.02$ unnamed
badge_dropdown 💀 $0.16$ $-0.16$ shinytest2_badge_dropdown_is_visible_when_clicking_on_it_multiple_times

Results for commit c330e41

♻️ This comment has been updated with latest results.

@osenan
Copy link
Copy Markdown
Contributor Author

osenan commented May 6, 2026

This is not required anymore thanks to this PR.

@osenan osenan closed this May 6, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant