Skip to content

normalize-sharing-prompts-during-iteration#1729

Merged
SteveSandersonMS merged 3 commits into
mainfrom
edburns/normalize-sharing-prompts-during-iteration
Jun 24, 2026
Merged

normalize-sharing-prompts-during-iteration#1729
SteveSandersonMS merged 3 commits into
mainfrom
edburns/normalize-sharing-prompts-during-iteration

Conversation

@edburns

@edburns edburns commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

.github/workflows/block-remove-before-merge.yml

I find it useful to store prompt and design files in a directory such as 1682-java-tool-ergonomics-prompts-remove-before-merge and have them live on the topic branch during iteration. But I never want to accidentally have those files merged.

This workflow prevents such accidental merges from happening.

.github/workflows/block-remove-before-merge.yml

I find it useful to store prompt and design files in a directory such as `1682-java-tool-ergonomics-prompts-remove-before-merge` and have them live on the topic branch during iteration. But I never want to accidentally have those files merged.

This workflow prevents such accidental merges from happening.
@edburns edburns requested a review from a team as a code owner June 18, 2026 22:28
Copilot AI review requested due to automatic review settings June 18, 2026 22:28
@edburns edburns requested a review from SteveSandersonMS June 18, 2026 22:29
Comment thread .github/workflows/block-remove-before-merge.yml Fixed

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

Adds a GitHub Actions workflow guard to prevent accidental merges of “work-in-progress” directories that are intended to be removed before merging (matching *-remove-before-merge/).

Changes:

  • Introduces a new PR-check workflow that enumerates changed files in a PR via gh api.
  • Fails the workflow with an error when any file path is under a *-remove-before-merge directory.
Show a summary per file
File Description
.github/workflows/block-remove-before-merge.yml New workflow that blocks PRs containing paths under *-remove-before-merge/.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread .github/workflows/block-remove-before-merge.yml
Comment thread .github/workflows/block-remove-before-merge.yml
edburns and others added 2 commits June 18, 2026 18:42
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@SteveSandersonMS SteveSandersonMS merged commit 079c558 into main Jun 24, 2026
11 checks passed
@SteveSandersonMS SteveSandersonMS deleted the edburns/normalize-sharing-prompts-during-iteration branch June 24, 2026 14:04
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.

4 participants