Skip to content

Revise unattended work instructions in copilot-instructions.md#15532

Merged
Evangelink merged 2 commits into
microsoft:mainfrom
nohwnd:update-copilot-instructions-v2
Mar 20, 2026
Merged

Revise unattended work instructions in copilot-instructions.md#15532
Evangelink merged 2 commits into
microsoft:mainfrom
nohwnd:update-copilot-instructions-v2

Conversation

@nohwnd
Copy link
Copy Markdown
Member

@nohwnd nohwnd commented Mar 20, 2026

Revised unattended work instructions based on review feedback on #15531:

  • Don't require Release builds locally — only as a troubleshooting step
  • Use --force-with-lease instead of plain force-push
  • Check git remote -v to identify fork vs upstream
  • Check PR status within 15-20 min (Windows finishes first)
  • Reason about automated review comments (they can be wrong)
  • Don't create draft PRs (undrafting forces re-build)
  • Mark green PRs as ready to merge

Clarifies #15531

- Don't require Release builds locally (only as troubleshooting step)
- Use --force-with-lease instead of plain force-push
- Check remote URLs to identify fork vs upstream
- Check PR status within 15-20 min (Windows finishes first)
- Reason about automated review comments
- Don't create draft PRs (undrafting forces re-build)
- Mark green PRs as ready to merge

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 20, 2026 09:07
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 unattended workflow guidance for autonomous work to reduce unnecessary local friction and improve PR hygiene/safety based on feedback from #15531.

Changes:

  • Loosens local build requirements (Debug OK) and adds Release build as a targeted troubleshooting step.
  • Improves PR workflow guidance (identify fork vs upstream, avoid draft PRs, check status earlier, mark ready-to-merge).
  • Recommends safer force-push behavior with --force-with-lease and calls out automated review comments as potentially incorrect.

Comment thread .github/copilot-instructions.md
Comment thread .github/copilot-instructions.md
Comment thread .github/copilot-instructions.md Outdated
@nohwnd nohwnd added the 🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it". label Mar 20, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Evangelink Evangelink merged commit bfd7e47 into microsoft:main Mar 20, 2026
2 checks passed
This was referenced May 26, 2026
This was referenced May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚢 Ship it! Add to PRs where owner approves automated PR, but cannot approve because they "wrote it".

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants