Skip to content

Remove stale github/ directory and add CI guard#698

Merged
strawgate merged 1 commit intomainfrom
fix/remove-stale-github-dir
Mar 14, 2026
Merged

Remove stale github/ directory and add CI guard#698
strawgate merged 1 commit intomainfrom
fix/remove-stale-github-dir

Conversation

@strawgate
Copy link
Collaborator

@strawgate strawgate commented Mar 14, 2026

Summary

  • Removes duplicate workflow prompt files from github/workflows/:
    • gh-aw-duplicate-issue-detector.md
    • gh-aw-fragments/safe-output-push-to-pr.md
    • gh-aw-stale-issues-investigator.md
    • gh-aw-stale-issues-remediator.md
  • Adds a new no-dotless-github CI job to .github/workflows/ci.yml that fails if a top-level github/ directory exists and prints offending files.

Why

The canonical workflow content is under .github/. This change removes duplicate copies and adds a CI guard so the dotless directory does not reappear.

Test plan

  • CI passes, including the new no-dotless-github job
  • No github/ directory exists in the repository root
  • Existing workflows remain unaffected apart from the new guard job

The body of this PR is automatically managed by the Trigger Update PR Body workflow.

The github/ directory (without dot prefix) contained outdated copies of
files that already exist in .github/. All four files are strictly older
than their .github/ counterparts — nothing to merge back.

Adds a CI job that fails if a github/ directory ever reappears.

Made-with: Cursor
@coderabbitai
Copy link

coderabbitai bot commented Mar 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 60435f3b-da47-474a-b815-81acec4a1266

📥 Commits

Reviewing files that changed from the base of the PR and between cf0e231 and e7302d0.

📒 Files selected for processing (5)
  • .github/workflows/ci.yml
  • github/workflows/gh-aw-duplicate-issue-detector.md
  • github/workflows/gh-aw-fragments/safe-output-push-to-pr.md
  • github/workflows/gh-aw-stale-issues-investigator.md
  • github/workflows/gh-aw-stale-issues-remediator.md
💤 Files with no reviewable changes (4)
  • github/workflows/gh-aw-stale-issues-remediator.md
  • github/workflows/gh-aw-fragments/safe-output-push-to-pr.md
  • github/workflows/gh-aw-duplicate-issue-detector.md
  • github/workflows/gh-aw-stale-issues-investigator.md

📝 Walkthrough

Walkthrough

This PR adds a validation job to .github/workflows/ci.yml that rejects top-level "github" directories without dot prefix. Concurrently, it deletes five workflow documentation files: gh-aw-duplicate-issue-detector.md, gh-aw-stale-issues-investigator.md, gh-aw-stale-issues-remediator.md, the safe-output-push-to-pr workflow fragment, and associated metadata and step definitions within them.

Possibly related PRs

Suggested labels

small_boom

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/remove-stale-github-dir
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

@strawgate strawgate merged commit 5c3bbdc into main Mar 14, 2026
16 of 17 checks passed
@strawgate strawgate deleted the fix/remove-stale-github-dir branch March 14, 2026 18:30
@github-actions github-actions bot added the medium_boom Medium PR blast radius; likely benefits from human review label Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium_boom Medium PR blast radius; likely benefits from human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant