Skip to content

paths-ignore on ci.yml means a docs-only or .conductor-only PR never reports the required 'test' check, so it cannot merge without --admin #236

Description

@cfdude

0.47.0 added paths-ignore ('.md', 'docs/', 'openspec/', '.conductor/') to ci.yml to save Actions minutes. Branch protection on main requires the 'test' check. A PR touching only ignored paths (e.g. #235, a CHANGELOG repair plus the post-release conductor upgrade) never starts the workflow, so 'test' never reports and the PR is unmergeable without an admin override.

Fix options: (a) drop paths-ignore and rely on concurrency+cancel-in-progress; (b) the documented GitHub pattern — a second workflow with the inverse paths filter that reports a passing 'test' job; (c) make the aggregate 'test' job run always and skip the matrix when only ignored paths changed (dorny/paths-filter or a git diff step).

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions