Skip to content

Avoid disabling inactive Step 0 workflow - #74

Merged
arilivigni merged 1 commit into
mainfrom
arilivigni-fix-workflow-reset
Aug 9, 2026
Merged

Avoid disabling inactive Step 0 workflow#74
arilivigni merged 1 commit into
mainfrom
arilivigni-fix-workflow-reset

Conversation

@arilivigni

Copy link
Copy Markdown
Collaborator

The Step 0 handoff still fails because the reusable start_exercise job has already disabled every exercise workflow, and gh workflow disable returns HTTP 403 when Step 0 is inactive.

Rely on the completed toolkit reset for Steps 0, 2, and 3, then use the gh workflow client only to enable Step 1 by filename.

The reusable start-exercise job already disables Steps 0, 2, and 3. Only enable Step 1 afterward to avoid the CLI's HTTP 403 for already-inactive workflows.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d523f2fa-5ffe-4981-846a-3627b824bae9
Copilot AI lite review requested due to automatic review settings August 9, 2026 17:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a Step 0 handoff failure by removing an attempt to disable the already-disabled Step 0 workflow (which can return HTTP 403 when inactive), while still enabling Step 1 so the exercise can proceed.

Changes:

  • Removes gh workflow disable "0-start-exercise.yml" from Step 0 to avoid failing when the workflow is inactive.
  • Keeps Step 0 responsible only for enabling Step 1 by workflow filename via gh workflow enable "1-step.yml".
Show a summary per file
File Description
.github/workflows/0-start-exercise.yml Stops disabling Step 0 (avoids 403 failure) and only enables Step 1 after the toolkit reset disables numbered workflows.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

@arilivigni
arilivigni merged commit 57dcb49 into main Aug 9, 2026
1 check passed
@arilivigni
arilivigni deleted the arilivigni-fix-workflow-reset branch August 9, 2026 17:49
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