Skip to content

ci: close a parent issue when all its sub-issues are closed (job in rbs.yml) - #50

Closed
Trosper3 wants to merge 3 commits into
ReduxISU:mainfrom
Trosper3:ci/close-finished-parents
Closed

Trosper3 wants to merge 3 commits into
ReduxISU:mainfrom
Trosper3:ci/close-finished-parents

Conversation

@Trosper3

@Trosper3 Trosper3 commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Lets a parent issue in this repo close itself once all its sub-issues are closed, using the shared workflow in Redux_Build_System. It's a second job in rbs.yml, so no workflow file is added.

Features

  • rbs.yml gains a close-parents job that calls ReduxISU/Redux_Build_System/.github/workflows/close-finished-parents.yml@main. It closes this repo's open issues labeled close-when-subissues-done once all their sub-issues are closed. Unlabeled issues, trackers included, are never touched.
  • New triggers: an issue closing, hourly (for sub-issues in other repos), and on demand. These run only close-parents. build-test still runs only on pushes and PRs.
  • The concurrency group adds the event name, so an hourly run can't cancel a push run on the same branch.

Same change as ReduxISU/Redux#950. Merge ReduxISU/Redux_Build_System#10 first: this calls its @main.

Trosper3 and others added 3 commits October 27, 2026 21:38
… rbs.yml

Adds a close-parents job that calls the shared close-finished-parents workflow in
Redux_Build_System on an issue closing and hourly. build-test still runs only on
pushes and PRs, so no workflow file is added.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Trosper3

Copy link
Copy Markdown
Contributor Author

The rbs check fails at startup ("workflow file issue") until ReduxISU/Redux_Build_System#10 merges. GitHub checks every workflow rbs.yml calls before starting a run, and close-finished-parents.yml isn't on rbs main yet. I confirmed the file is valid by pointing it at the rbs PR branch for one run on #50: build-test ran and close-parents was skipped, as intended. Then I reverted that. After #10 merges, re-run the rbs check here.

@Trosper3

Copy link
Copy Markdown
Contributor Author

Marked as draft until ReduxISU/Redux_Build_System#10 merges. Merging this first would make every rbs run in this repo fail at startup, because the shared workflow it calls wouldn't exist yet. Once #10 is in, I'll mark this ready and re-run the check.

@Trosper3
Trosper3 marked this pull request as ready for review October 28, 2026 05:08
@Trosper3 Trosper3 closed this Oct 28, 2026
@Trosper3 Trosper3 reopened this Oct 28, 2026
@Trosper3

Trosper3 commented Nov 4, 2026

Copy link
Copy Markdown
Contributor Author

Closing this: we decided to close parent issues by hand, since each one gets checked before it closes anyway. Removing the automatic job everywhere instead (Redux_Frontend and Redux_Build_System get small removal PRs).

@Trosper3 Trosper3 closed this Nov 4, 2026
@Trosper3
Trosper3 deleted the ci/close-finished-parents branch November 4, 2026 03:17
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.

1 participant