Skip to content

THROWAWAY treatment for #2767 — do not merge - #3304

Closed
kovtcharov-amd wants to merge 2 commits into
tmp/2767-verify-basefrom
tmp/2767-verify-head
Closed

THROWAWAY treatment for #2767 — do not merge#3304
kovtcharov-amd wants to merge 2 commits into
tmp/2767-verify-basefrom
tmp/2767-verify-head

Conversation

@kovtcharov-amd

Copy link
Copy Markdown
Collaborator

Throwaway. Treatment arm for #2767: the same stacked PR WITH the fix applied. Expect the email/unit/lint checks to appear. Closing shortly.

Ovtcharov added 2 commits September 2, 2026 09:15
`on.pull_request.branches` matches the PR's BASE branch, not its head, so
`branches: [ main ]` meant "only when merging into main". Every PR stacked on
a feature branch fell outside the filter and the jobs were never created —
nothing failed, nothing showed as skipped, the checks page was simply missing
them. #2599 merged email-agent changes with zero email tests, zero unit tests
and zero lint behind a green page.

Drops the filter from the 34 workflows that carried it. `paths:` filters and
`push:` triggers are untouched, so a workflow still only runs for the code it
covers and still only runs on main for pushes.

`util/check_workflow_triggers.py` (wired into `lint.py --all`) fails the build
if the filter comes back; lint.yml now watches `.github/workflows/**` so a
workflows-only PR actually runs that gate.
@github-actions github-actions Bot added documentation Documentation changes devops DevOps/infrastructure changes tests Test changes security Security-sensitive changes cpp agent::email Email agent changes labels Sep 2, 2026
@kovtcharov-amd

Copy link
Copy Markdown
Collaborator Author

Closing — throwaway verification PR, not intended to merge. The #2767 investigation it belonged to was stopped: stacked-branch CI coverage is by design, so there is nothing to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent::email Email agent changes cpp devops DevOps/infrastructure changes documentation Documentation changes security Security-sensitive changes tests Test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant