Skip to content

ci: run PR validation for feature/* base branches - #1586

Merged
Aaronontheweb merged 1 commit into
netclaw-dev:devfrom
Aaronontheweb:ci/feature-branch-pr-validation
Jul 5, 2026
Merged

ci: run PR validation for feature/* base branches#1586
Aaronontheweb merged 1 commit into
netclaw-dev:devfrom
Aaronontheweb:ci/feature-branch-pr-validation

Conversation

@Aaronontheweb

Copy link
Copy Markdown
Collaborator

Summary

Add feature/* base-branch support to PR-validation workflows so feature branches (e.g. feature/memory-embeddings) run full CI validation on their own PRs.

GitHub's pull_request and push branch filters match the base branch and branch head respectively. Without explicit feature/* entries, feature-targeted PRs were skipping validation.

Workflows Modified

Workflow Trigger Change Reason
pr_validation.yml push + pull_request Added feature/* Unit tests, slopwatch, copyright checks
smoke.yml push + pull_request Added feature/* Native end-to-end smoke harness
validate_docker_image.yml push + pull_request Added feature/* Docker build validation (paths filter preserved)

Workflows Unchanged

Workflow Trigger Reason
demo_smoke.yml workflow_dispatch Manual-only, not PR gating
publish_release_binaries.yml push (tags) Release workflow, tag-triggered
publish_skills.yml workflow_call + workflow_dispatch Publishing only, called from publish_release_binaries
website-rebuild.yml release (published) Publishing only, not validation

YAML Validation

All modified workflows pass Python yaml.safe_load() validation.

Add `feature/*` to PR-validation workflow base-branch filters.

GitHub's pull_request and push branches filters match the BASE branch and
branch head, respectively. Without explicit `feature/*` filters, PRs targeting
`feature/*` branches (e.g., `feature/memory-embeddings`) were skipping full
CI validation.

This change enables the embedding stack and future feature branches to run
full validation on their own PRs while maintaining the current dev/master/main
flow. Release and publish workflows are unchanged.

Workflows modified:
- pr_validation.yml: added feature/* to push and pull_request
- smoke.yml: added feature/* to push and pull_request
- validate_docker_image.yml: added feature/* to push and pull_request (with
  existing paths filter)

Workflows skipped (unchanged):
- demo_smoke.yml: workflow_dispatch only (manual trigger)
- publish_release_binaries.yml: tag-triggered release workflow
- publish_skills.yml: workflow_call + workflow_dispatch (publishing)
- website-rebuild.yml: release-triggered (publishing)
@Aaronontheweb
Aaronontheweb merged commit ef30b02 into netclaw-dev:dev Jul 5, 2026
15 checks passed
@Aaronontheweb
Aaronontheweb deleted the ci/feature-branch-pr-validation branch July 5, 2026 20:03
Aaronontheweb added a commit that referenced this pull request Jul 5, 2026
… capability-provenance logging #1584, FakeChatClient thread-safety #1579)
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