Repository navigation
Remove retired data snapshot rollout flag - #471
Merged
Merged
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8fdf6fe0-0e03-49a2-ad0c-f84786acfdd8
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The scoped cleanup preserves snapshot safeguards, release selection, and operational controls, with no blocking issues identified.
0 open findings
What changed in this PR
Removes the retired DATA_BRANCH_SNAPSHOTS workflow injection while preserving permanent snapshot safety and processor environment inheritance coverage.
Changes:
- Removes obsolete flag injection from dashboard and collector workflows.
- Updates regression tests for flag removal and unset-variable behavior.
- Documents post-merge variable deletion and restrictions on older processors.
| File | Description |
|---|---|
| pull-request-dashboard/README.md | Documents obsolete variable cleanup. |
| github-actions-queue/README.md | Clarifies unconditional snapshot storage. |
| .github/workflows/pull-request-dashboard-refresh-author-nudges.yml | Removes flag injection from reminder sweeps. |
| .github/workflows/pull-request-dashboard-drain.yml | Removes flag injection from queue draining. |
| .github/workflows/github-actions-queue-collector.yml | Removes flag injection from data operations. |
| .github/scripts/pull-request-dashboard/WEBHOOK_SETUP.md | Updates cleanup guidance and preserves rollback restrictions. |
| .github/scripts/pull-request-dashboard/test_rollout.py | Checks workflows no longer reference the flag. |
| .github/scripts/pull-request-dashboard/test_owned_execution.py | Preserves generic environment inheritance coverage. |
| .github/scripts/pull-request-dashboard/test_data_snapshots.py | Adds snapshot coverage with the flag unset. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Pull request dashboard statusMerged · refreshed 2026-10-14 17:37 UTC Status above doesn't look right?
|
Kielek
approved these changes
Oct 14, 2026
codeboten
approved these changes
Oct 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Remove the remaining
DATA_BRANCH_SNAPSHOTSworkflow injection now that unconditional snapshot storage is released and promoted and older writers have retired.