Skip to content

Add junco back to revdepcheck inputs#281

Merged
shajoezhu merged 5 commits into
mainfrom
copilot/add-junco-back-to-reverse-dep-check
Jul 13, 2026
Merged

Add junco back to revdepcheck inputs#281
shajoezhu merged 5 commits into
mainfrom
copilot/add-junco-back-to-reverse-dep-check

Conversation

Copilot AI commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

junco was missing from the reverse dependency check inputs, so scheduled revdep runs no longer covered it. This updates the revdep package list so junco is included again.

  • Reverse dependency coverage

    • Added johnsonandjohnson/junco to .revdeprefs.yaml, which defines the package refs consumed by the scheduled revdepcheck workflow.
  • Scope

    • No runtime/package code changes.
    • No workflow logic changes; this only restores the missing revdep target.
- insightsengineering/scda.test
- insightsengineering/chevron
- insightsengineering/autoslider.core
- insightsengineering/teal.modules.clinical
- johnsonandjohnson/junco

Copilot AI linked an issue Jul 13, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add junco back to reverse dep check Add junco back to revdepcheck inputs Jul 13, 2026
Copilot AI requested a review from shajoezhu July 13, 2026 11:35
@shajoezhu shajoezhu marked this pull request as ready for review July 13, 2026 11:44
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files    5 suites   7s ⏱️
 51 tests  51 ✅ 0 💤 0 ❌
182 runs  182 ✅ 0 💤 0 ❌

Results for commit c7549eb.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                 Stmts    Miss  Cover    Missing
---------------------  -------  ------  -------  ----------------------------------------------------------
R/paginate_listing.R        29       0  100.00%
R/rlistings_methods.R      140      17  87.86%   40-41, 43, 73, 92, 96, 196-199, 202, 286-292
R/rlistings.R              367      25  93.19%   237-244, 370, 379, 460, 644-647, 651-654, 683-684, 725-728
TOTAL                      536      42  92.16%

Diff against main

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

Results for commit: c7549eb

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@shajoezhu shajoezhu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@shajoezhu shajoezhu enabled auto-merge (squash) July 13, 2026 12:20
@shajoezhu shajoezhu merged commit c7549eb into main Jul 13, 2026
29 checks passed
@shajoezhu shajoezhu deleted the copilot/add-junco-back-to-reverse-dep-check branch July 13, 2026 12:21
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 13, 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.

add junco back to reverse dep check

2 participants