Skip to content

infra: split CI jobs, upgrade Node 22, add branch protection#11

Merged
ptheofan merged 7 commits intomainfrom
infra/ci-block-bad-pr
Apr 11, 2026
Merged

infra: split CI jobs, upgrade Node 22, add branch protection#11
ptheofan merged 7 commits intomainfrom
infra/ci-block-bad-pr

Conversation

@ptheofan
Copy link
Copy Markdown
Owner

Summary

  • Split monolithic test job into parallel Lint, Type Check, and Tests jobs for clearer PR status checks
  • Build job now depends on all three passing (needs: [lint, typecheck, test])
  • Upgraded Node.js from 20 to 22 to resolve GitHub Actions deprecation warnings
  • Added branch protection rule on main requiring all 4 CI checks to pass before merge

Test plan

  • Verify CI runs all 4 jobs on this PR
  • Confirm branch protection blocks merge if any job fails

🤖 Generated with Claude Code

ptheofan and others added 7 commits March 9, 2026 17:10
One-way markdown → Google Docs sync with three-way diffing
infrastructure for surgical updates that preserve reviewer comments.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
16-task plan covering auth, converter, three-way diffing, API wrapper,
IPC handlers, UI components, mermaid support, and E2E tests.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Split monolithic test job into parallel Lint, Type Check, and Tests
jobs for clearer PR status checks. Build job now depends on all three
passing. Upgraded Node.js from 20 to 22 to resolve GitHub Actions
deprecation warnings. Branch protection rule added requiring all CI
checks to pass before merging to main.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@ptheofan ptheofan merged commit e65c97d into main Apr 11, 2026
4 checks passed
@ptheofan ptheofan deleted the infra/ci-block-bad-pr branch April 11, 2026 14:46
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