Skip to content

Add macros submodule, open PR in d-morrison/macros, and checkout submodule in Copilot setup - #18

Merged
d-morrison merged 20 commits into
mainfrom
copilot/add-submodule-macros
Apr 22, 2026
Merged

Add macros submodule, open PR in d-morrison/macros, and checkout submodule in Copilot setup#18
d-morrison merged 20 commits into
mainfrom
copilot/add-submodule-macros

Conversation

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Edits the macros submodule to add missing summary statistic macros, opens a pull request for those changes in the upstream d-morrison/macros repository, and configures the Copilot setup workflow to check out the submodule.

Changes Made

  • Macros submodule: Added correlation (\Cor, \Corf, \Cors, \Cort), median (\med, \medf), and IQR (\IQR, \IQRf) macros to macros.qmd, following the same naming conventions as the existing \Var, \Cov, and \SD macros.
  • Branch pushed: copilot/add-summary-stat-macros pushed to d-morrison/macros.
  • PR opened: d-morrison/macros#22 — "feat: add correlation, median, and IQR macros" from copilot/add-summary-stat-macros into main.
  • Copilot setup: Added submodules: true to the actions/checkout@v4 step in copilot-setup-steps.yml so the macros submodule is initialized and checked out during Copilot agent environment setup.

Copilot AI linked an issue Apr 21, 2026 that may be closed by this pull request
Copilot AI and others added 5 commits April 21, 2026 02:19
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/a35eba7b-1ce6-4ccd-b401-6e268da8eb53

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/a35eba7b-1ce6-4ccd-b401-6e268da8eb53

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/a35eba7b-1ce6-4ccd-b401-6e268da8eb53

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/a35eba7b-1ce6-4ccd-b401-6e268da8eb53

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot AI changed the title [WIP] Add macros repository as a submodule Add macros submodule and enable Copilot cross-repo auth for PRs Apr 21, 2026
Copilot AI requested a review from d-morrison April 21, 2026 02:24
@d-morrison

Copy link
Copy Markdown
Collaborator

@copilot try using MACROS_REPO_PAT to push commits to https://github.com/d-morrison/macros

Copilot AI and others added 3 commits April 21, 2026 03:39
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/0745a909-6e25-426a-a4b7-b20096c7613a

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/0745a909-6e25-426a-a4b7-b20096c7613a

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/0745a909-6e25-426a-a4b7-b20096c7613a

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

@copilot try using MACROS_REPO_PAT to push commits to https://github.com/d-morrison/macros

Implemented. The Copilot setup workflow now uses MACROS_REPO_PAT to authenticate and explicitly verify write access to https://github.com/d-morrison/macros during setup (gh api repos/d-morrison/macros --jq '.permissions.push'). Commit: c62fa5d.

@d-morrison

Copy link
Copy Markdown
Collaborator

@copilot create a PR in https://github.com/d-morrison/macros

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

@d-morrison I've opened a new pull request, #21, to work on those changes. Once the pull request is ready, I'll request review from you.

@d-morrison
d-morrison marked this pull request as ready for review April 21, 2026 18:29
Comment thread .github/workflows/copilot-setup-steps.yml

Copilot AI commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

@copilot create a PR in https://github.com/d-morrison/macros

PR d-morrison/macros#22 is open — "feat: add correlation, median, and IQR macros" from branch copilot/add-summary-stat-macros into main.

Copilot AI changed the title Add macros submodule and enable Copilot cross-repo auth for PRs Add macros submodule and open PR in d-morrison/macros Apr 21, 2026
Copilot AI requested a review from d-morrison April 21, 2026 18:49
@d-morrison

Copy link
Copy Markdown
Collaborator

@copilot checkout macros submodule in copilot-startup-steps

Agent-Logs-Url: https://github.com/UCD-SERG/qwt/sessions/faab53d3-d075-4415-8c98-7315edec6e4b

Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>

Copilot AI commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

@copilot checkout macros submodule in copilot-startup-steps

Done in commit c096d06. Added submodules: true to the actions/checkout@v4 step so the macros submodule is initialized and checked out during Copilot setup.

Copilot AI changed the title Add macros submodule and open PR in d-morrison/macros Add macros submodule, open PR in d-morrison/macros, and checkout submodule in Copilot setup Apr 22, 2026
@d-morrison
d-morrison merged commit 72a0cf2 into main Apr 22, 2026
8 checks passed
@d-morrison
d-morrison deleted the copilot/add-submodule-macros branch April 22, 2026 01:13
@github-actions

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-04-22 01:15 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add submodule

2 participants