Skip to content

fix: update course-updater to target main instead of staged#25

Merged
DanWahlin merged 1 commit intomainfrom
fix/course-updater-target-main
Mar 31, 2026
Merged

fix: update course-updater to target main instead of staged#25
DanWahlin merged 1 commit intomainfrom
fix/course-updater-target-main

Conversation

@DanWahlin
Copy link
Copy Markdown
Collaborator

Updates the course-updater workflow to target main instead of the deleted staged branch. The PR review step already serves as the review gate, making staged unnecessary.

Copilot AI review requested due to automatic review settings March 31, 2026 19:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the course-updater agentic workflow configuration to open pull requests against main now that the staged branch has been removed.

Changes:

  • Switch create-pull-request.base-branch from staged to main in the workflow source markdown.
  • Recompile the generated workflow (course-updater.lock.yml) so safe-outputs config and checkout ref target main.
  • Add a new .github/uv-stats.csv file (appears unrelated to the stated PR purpose).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/course-updater.md Updates workflow frontmatter and instructions to target main.
.github/workflows/course-updater.lock.yml Regenerated compiled workflow reflecting main as the base/checkout branch.
.github/uv-stats.csv New CSV data file added; no references found in repo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +1 to +5
"03/03",197
"03/04",189
"03/05",349
"03/06",359
"03/07",157
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

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

This new CSV file appears unrelated to the workflow base-branch change described in the PR, and it is not referenced anywhere in the repo (no code/docs/workflows mention uv-stats). If it’s accidental generated data, please remove it; if it’s intended, add a header/description and a clear consumer (doc or workflow) so its purpose and update process are explicit.

Copilot uses AI. Check for mistakes.
@DanWahlin DanWahlin self-assigned this Mar 31, 2026
@DanWahlin DanWahlin merged commit 2a2f818 into main Mar 31, 2026
7 checks passed
@DanWahlin DanWahlin deleted the fix/course-updater-target-main branch March 31, 2026 20:00
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.

2 participants