Skip to content

Fix copilot-setup-steps: replace reusable workflow with direct steps#388

Merged
m7pr merged 1 commit into
mainfrom
fix/copilot-setup-steps-direct-steps
Dec 15, 2025
Merged

Fix copilot-setup-steps: replace reusable workflow with direct steps#388
m7pr merged 1 commit into
mainfrom
fix/copilot-setup-steps-direct-steps

Conversation

@m7pr

@m7pr m7pr commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

This PR fixes the GitHub Copilot setup workflow by replacing the reusable workflow call with direct steps, as required by GitHub Copilot's limitations for copilot-setup-steps jobs.

Changes:

  • Removed uses: insightsengineering/r.pkg.template/.github/workflows/copilot-setup-steps.yaml@main
  • Added direct steps for checkout, R setup, and dependency installation
  • Uses actions with uses: (which is allowed) instead of reusable workflows (which are not)

This resolves the error: 'Calling a reusable workflow with the uses: keyword is not supported in copilot-setup-steps jobs.'

@github-actions

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------
R/cdisc_data.R                   1       0  100.00%
R/deprecated.R                  18       0  100.00%
R/dummy_function.R               2       2  0.00%    15-16
R/formatters_var_labels.R       61       0  100.00%
R/join_key.R                    38       0  100.00%
R/join_keys-c.R                 12       0  100.00%
R/join_keys-extract.R          128       0  100.00%
R/join_keys-names.R             15       0  100.00%
R/join_keys-parents.R           30       0  100.00%
R/join_keys-print.R             45       0  100.00%
R/join_keys-utils.R             73       3  95.89%   35-38
R/join_keys.R                   21       0  100.00%
R/teal_data-class.R             36       6  83.33%   50-52, 59, 104-105
R/teal_data-constructor.R       11       2  81.82%   48, 51
R/teal_data-extract.R            3       0  100.00%
R/teal_data-get_code.R          13       8  38.46%   114-120, 124
R/teal_data-names.R              8       1  87.50%   42
R/teal_data-show.R               5       5  0.00%    14-20
R/topological_sort.R            32       0  100.00%
R/verify.R                      42      11  73.81%   67, 97-101, 104-108
TOTAL                          594      38  93.60%

Diff against main

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

Results for commit: 2328ed1

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   15 suites   2s ⏱️
163 tests 163 ✅ 0 💤 0 ❌
218 runs  218 ✅ 0 💤 0 ❌

Results for commit 2328ed1.

♻️ This comment has been updated with latest results.

@m7pr
m7pr merged commit 847f297 into main Dec 15, 2025
26 checks passed
@m7pr
m7pr deleted the fix/copilot-setup-steps-direct-steps branch December 15, 2025 12:39
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 15, 2025
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