Skip to content

Add conditional check for auto update - #820

Merged
MattyTheHacker merged 1 commit into
mainfrom
fix-auto-update
Aug 9, 2026
Merged

Add conditional check for auto update#820
MattyTheHacker merged 1 commit into
mainfrom
fix-auto-update

Conversation

@MattyTheHacker

Copy link
Copy Markdown
Member

Turns out it wasn't even my action causing the bug, it was the step before that generated the access token, so we need to add a fork check to the start of the entire workflow.

Signed-off-by: Matty Widdop <18513864+MattyTheHacker@users.noreply.github.com>
@MattyTheHacker MattyTheHacker self-assigned this Aug 6, 2026
@MattyTheHacker
MattyTheHacker requested review from a team and a lite review from Copilot August 6, 2026 23:33
@MattyTheHacker MattyTheHacker added the bug Something isn't working label Aug 6, 2026
@MattyTheHacker MattyTheHacker added the deployment Changes to the deployment or CI/CD configuration label Aug 6, 2026

This comment was marked as low quality.

@codecov

codecov Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@CarrotManMatt CarrotManMatt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That would make sense to have the checks on the entire workflow👍🏻

@MattyTheHacker MattyTheHacker added the awaiting committee review PRs awaiting approval from a CSS committee member label Aug 7, 2026
@MattyTheHacker
MattyTheHacker merged commit 55ff8a1 into main Aug 9, 2026
14 checks passed
@MattyTheHacker
MattyTheHacker deleted the fix-auto-update branch August 9, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting committee review PRs awaiting approval from a CSS committee member bug Something isn't working deployment Changes to the deployment or CI/CD configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants