Skip to content

ci: remove Linear Issue Reference gate from validate.yml#225

Merged
amcheste merged 2 commits into
developfrom
ci/remove-linear-ref-check
May 11, 2026
Merged

ci: remove Linear Issue Reference gate from validate.yml#225
amcheste merged 2 commits into
developfrom
ci/remove-linear-ref-check

Conversation

@amcheste-ai-agent
Copy link
Copy Markdown
Contributor

Summary

Removes the linear-ref job from .github/workflows/validate.yml. Linear is used for major project tracking only, not every PR — requiring an AMC-N reference (or the No-Linear-Issue: opt-out trailer) on every PR adds friction without proportional value.

Why

The check was added after AMC-71/72/73/74 stayed in Backlog post-merge (verified 2026-05-02, per the workflow's own comment). That failure mode is real for tickets that should have been referenced. But:

  • Linear is now used selectively, not for every PR's worth of work.
  • The opt-out trailer (No-Linear-Issue: <reason>) made every untracked PR a ritualistic two-line affair.
  • Bot-authored PRs (e.g., this one's sibling PR chore(brand): em-dash sweep across prose + remove submitted CFP draft #224) categorically don't have tickets — the trailer becomes muscle memory rather than information.

After this change: PRs working a Linear ticket will still reference it (you do that naturally for the auto-close); PRs that don't need a ticket no longer have to declare that they don't.

What's removed

  • The linear-ref job (49 lines): comment block + job definition.

What's NOT removed

  • All other validation jobs: Lint, Unit & Integration Tests, Build Operator Image, Commit Lint, Detect Changed Files, Semgrep, Update Release Draft. Same triggers, same enforcement.
  • The Linear ↔ GitHub sync itself (configured outside this repo). PRs that reference Linear tickets will still auto-close on merge; the sync doesn't depend on this CI check.

Verification

  • grep -c "linear-ref\|Linear Issue Reference\|AMC-" .github/workflows/validate.yml → 0.
  • No other workflow files reference linear-ref or its outputs.
  • File ends cleanly (last line: } | tee -a "$GITHUB_STEP_SUMMARY" from the prior release-bump-suggestion job).

Note

This PR includes a No-Linear-Issue: trailer on the commit so the gate that's being removed allows this very PR through to merge. After merge, the trailer becomes vestigial — future PRs don't need it.

🤖 Generated with Claude Code

No-Linear-Issue: removing the gate itself, the gate would be self-referentially required to allow this PR to merge

Linear is used for major project tracking only, not every PR.
Requiring an AMC-N reference (or the No-Linear-Issue opt-out
trailer) on every PR adds friction without proportional value
for the work that doesn't map to a Linear ticket. Particularly
ritualistic for bot-authored PRs, which categorically don't
have tickets.

The original incident that prompted this check (AMC-71/72/73/74
staying in Backlog after their work shipped, 2026-05-02) is real,
but the failure mode is acceptable: Linear tickets for major
projects that DO get referenced will still auto-close on merge.
Tickets that nobody referenced just need a manual close, which
is the same operation the team is already doing for tickets that
never had PRs.

Removes the `linear-ref` job (49 lines) from validate.yml. No
other changes; all other validation jobs (Lint, Tests,
Build Operator Image, Commit Lint, Detect Changed Files, Semgrep)
keep firing as before.

No-Linear-Issue: removing the gate itself, the gate would be self-referentially required to allow this PR to merge

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: amcheste <13696614+amcheste@users.noreply.github.com>
@amcheste amcheste merged commit bf1d0e5 into develop May 11, 2026
8 checks passed
@amcheste amcheste deleted the ci/remove-linear-ref-check branch May 11, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants