Skip to content

fix: restrict CI triggers to push-to-main and any PR#18

Merged
nsheaps merged 2 commits intomainfrom
claude/fix-ci-pr-workflow-XjOeV
Mar 5, 2026
Merged

fix: restrict CI triggers to push-to-main and any PR#18
nsheaps merged 2 commits intomainfrom
claude/fix-ci-pr-workflow-XjOeV

Conversation

@nsheaps
Copy link
Copy Markdown
Owner

@nsheaps nsheaps commented Mar 5, 2026

Summary

  • Push trigger: restricted from all branches ('*') to main only, eliminating duplicate CI runs on PR branches
  • PR trigger: removed explicit types filter so all PR event types are handled; no branch filter applied
  • Path filtering: added paths-ignore for both triggers to skip CI on docs-only and non-code changes (.md, docs/, .github/pages/, LICENSE, .vscode/)

Test plan

  • Verify CI runs on push to main
  • Verify CI runs on PR open/sync against any base branch
  • Verify CI does NOT run on push to non-main branches (outside of PRs)
  • Verify CI does NOT run on docs-only changes

https://claude.ai/code/session_01RcKeVetFxJCbWvySi24ZvP

Previously CI ran on push to all branches, causing duplicate runs on
PRs (once for push, once for pull_request). Now pushes only trigger on
main. PR trigger has no branch or event type filter. Both triggers
ignore docs-only and non-code changes.

https://claude.ai/code/session_01RcKeVetFxJCbWvySi24ZvP
@nsheaps nsheaps self-assigned this Mar 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Release Version Check

🟡 MINOR version bump

Version
Current 0.0.1
Next 0.1.0

This version will be released automatically when this PR is merged to main.

Changelog preview

0.1.0 (2026-03-05)

Features

  • add Claude Code rules for UI evidence, task completion, GitHub access, and PR management (4520912)
  • add file browser, space switcher, disable imports, fix base path detection (4113a81)
  • add path-based SPA routing with 404.html redirect hack (c73c62e), closes #pageId
  • add release-it versioning with conventional changelog (2e59640)
  • bdd: create features/ directory structure with feature files [T0.9] (e4cd58d)
  • ci: add GitHub Pages preview deployment workflow [T0.11] (f2670d8)
  • core: add all abstraction interfaces — types only [T0.3] (3d4e5b2)
  • core: add relations and rollups engine for database properties (T4.10) (8590b86)
  • core: add WebFsBackend using File System Access API [P2.7] (6308066)
  • deploy to GitHub Pages on push to main and add live demo link (da797f0)
  • e2e: set up Playwright with smoke test [T0.6] (f46a8f6)
  • editor: add table rendering and slash command [P2.4d] (c24de18)
  • infra: initialize monorepo with Nx, bun workspaces, TypeScript [T0.1] (0439217)
  • infra: set up CI workflow, ESLint, Prettier, typecheck [T0.2] (93e2f23)
  • restructure sidebar, improve page header, add trash page view (32b2e2f)
  • show changelog preview in PR version check comment (d010406)
  • show dynamic version on about page with commit SHA for preview deploys (e3d3874)
  • T5.1 — GitHub OAuth App flow with device flow support (e8856c8)
  • T5.2 — Repo picker/creator UI component (c851397)
  • T5.3 — Auto-commit engine with debounce and batching (370e446)
  • T5.4 — Auto-branch strategy (single, per-device, per-session) (e911198)
  • T5.5 — Auto-merge + conflict resolution engine (0269e42)
  • T5.6 — Conflict notification + resolution UI (e639977)
  • T5.7 — Page history viewer with Git log + diff (b1386e5)
  • T5.8 — Sync engine with pull/push cycle (06c110a)
  • test: set up BDD tooling with @amiceli/vitest-cucumber [T0.8] (9f1c8f0)
  • test: set up Vitest with unit test [T0.7] (bb84641)
  • ui: add DatabaseBoardView with drag-and-drop Kanban columns (T4.3) (b1b9da9)
  • ui: add DatabaseCalendarView with month grid and event display (T4.4) (6c9a714)
  • ui: add DatabaseGalleryView with image cards and property previews (T4.6) (00ca342)
  • ui: add DatabaseListView with expandable row details (T4.7) (c4c05ad)
  • ui: add DatabaseMapView with location parsing and marker list (T4.5) (934d2c8)
  • ui: add DatabaseProvider and useDatabaseEngine hook [P2.5] (8634ed4)
  • ui: add DatabaseTableView with sorting, filtering, and cell formatting (T4.2) (87195d2)
  • ui: add folder view for pages with children [P2.4b] (84bf88e)
  • ui: add import/export dialogs for Notion, Obsidian, and page export [P2.10-P2.12] (d2062ed)
  • ui: add InlineDatabase TipTap extension for embedded database blocks (T4.8) (fe9e602)
  • ui: add landing page for GitHub Pages with guides and demo info [P2.4a] (d2b6267)
  • ui: add LinkedDatabaseView with view type tabs and name editing (T4.9) (6fb79b6)
  • ui: add page cover banner and icon support [P2.4c] (35572d2)
  • ui: add PropertyEditor for all database property types (T4.12) (44bff25)
  • ui: add SelectEditor and MultiSelectEditor with color tags (T4.11) (772dd52)
  • ui: add SpaceManager for multi-space support [P2.9] (e06266f)
  • ui: create UI package skeleton with React, TailwindCSS [T0.4] (79972bf)
  • ui: implement CEPT_DEMO_MODE with sample content [T0.12] (202795d)
  • ui: replace localStorage with BrowserFsBackend (keystone P2.3) (2d857df)
  • ui: wire page CRUD to storage backend — individual page files [P2.4] (876e7da)
  • ui: wire search index to CeptSearchIndex with TF-IDF scoring [P2.6] (1cf10bd)
  • use release-it tag flow instead of push-to-main for deploys (2915703)
  • web: create Vite entry point with dev server [T0.5] (d3d758f)
  • wire spaces/import/export into UI, fix toggle touch, add responsive E2E (e0d4fbf)

Bug Fixes

  • add d3 as proper dependency for KnowledgeGraph component (4e496e2)
  • auto-release on merge to main and add PR version comments (5669874)
  • checkout PR head ref to avoid detached HEAD in version check (44d4e03)
  • CI mise version specs and vitest project configuration (99c879e)
  • CI runs on all branches, add e2e to workspaces (1258c4a)
  • core,signaling: fix GitBackend node:fs hardcode, diff empty hunks, signaling entry point [P2.1] (32cf46b)
  • core: harden Markdown parser/serializer roundtrip, fix extractText [P2.2] (c7cffa2)
  • deploy to /cept/app/ with site-level 404 router for SPA deep links (be6fefe)
  • guard URL updates behind initialization, fix reload deep linking (9497a5c)
  • page title hover-only outline, no selected bg in recent, rename Move... (225b3ac)
  • release workflow failures and set initial version to 0.0.1 (68c27cc)
  • remove unused variable 'i' in graph-animation test to fix lint (d288275)
  • restore @amiceli/vitest-cucumber dependency in package.json (5661f54)
  • restrict CI triggers to push-to-main and any PR (30535bf)
  • run vitest from root instead of per-package via NX (8835894)
  • update .claude/settings.json for unattended tool execution (c35b8b1)
  • web: stub node: imports in web build, update continue protocol (4d0799f)
  • wire up App integration layer, fix Tailwind build, fix layout (b7020f2)

Refactoring

  • DRY 404/index.html into shared .github/pages/ files (5c7461e)

Documentation

  • create feature spec stubs for Phase 1 features [T0.10] (4af7c9b)
  • mark P2.4f and P2.4g complete in TASKS.md [P2.4f, P2.4g] (313b9c4)
  • mark Phase 1 complete in TASKS.md (10b56d5)

Tests

  • editor: add table rendering tests and E2E coverage [P2.4e] (e29b842)

CI/CD

  • run CI on all pull requests regardless of base branch (e67a8fe)
  • run E2E tests on all branches and draft PRs (964da1b)

Chores

  • regenerate bun.lock after rebase (568ee3a)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Preview Deployment

The web app for this PR has been deployed:

Open Preview

Use this to verify the app works correctly, especially for dependency updates.

github-actions bot added a commit that referenced this pull request Mar 5, 2026
Updated CI workflow to simplify branch and path ignore settings.
@nsheaps nsheaps marked this pull request as ready for review March 5, 2026 20:15
@nsheaps nsheaps enabled auto-merge (squash) March 5, 2026 20:16
github-actions bot added a commit that referenced this pull request Mar 5, 2026
@nsheaps nsheaps merged commit 55ad125 into main Mar 5, 2026
8 checks passed
@nsheaps nsheaps deleted the claude/fix-ci-pr-workflow-XjOeV branch March 5, 2026 20:18
github-actions bot added a commit that referenced this pull request Mar 5, 2026
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.

2 participants