Skip to content

disable requireCleanGit for skill-optimizer CI runs#28378

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/skill-optimizer-improvements
Closed

disable requireCleanGit for skill-optimizer CI runs#28378
Copilot wants to merge 1 commit into
mainfrom
copilot/skill-optimizer-improvements

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 25, 2026

The daily skill-optimizer was failing with run_status: 1 because optimize.requireCleanGit blocked execution whenever the CI workspace had uncommitted files (e.g. from a prior make recompile or init step).

Changes

  • .skill-optimizer/skill-optimizer.jsonrequireCleanGit set to false, removing the clean-git guard so the optimizer proceeds regardless of workspace state
  • .github/workflows/daily-skill-optimizer.md — added git stash --include-untracked || true before the optimizer run and git stash pop || true in a post-run restore step as a belt-and-suspenders complement

Copilot AI linked an issue Apr 25, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Improve handling of uncommitted changes in skill optimizer disable requireCleanGit for skill-optimizer CI runs Apr 25, 2026
Copilot AI requested a review from pelikhan April 25, 2026 00:51
@pelikhan pelikhan closed this Apr 25, 2026
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.

[skill-optimizer] Daily Skill Optimizer Improvements - 2026-04-24

2 participants