Skip to content

[DO NOT MERGE] Test starflow PR 201 - #6496

Closed
lengau wants to merge 1 commit into
mainfrom
work/bot/starflow-test-pr-201
Closed

lengau wants to merge 1 commit into
mainfrom
work/bot/starflow-test-pr-201

Conversation

@lengau

@lengau lengau commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

This is an automated draft PR to test changes from starflow PR #201.

  • Starflow PR: feat: add test-pr skill for testing PRs across downstream repos starflow#201
  • Target Ref: canonical/starflow@work/testing-skill
  • Starflow Workflows Tested: policy.yaml, qa.yaml, tics.yaml
  • Disabled Non-Starflow Jobs (if: false): check-renovate.yaml (renovate), copilot-collections-update.yaml (check-update), copilot-setup-steps.yaml (copilot-setup-steps), publish.yaml (publish), qa.yaml (filter), release-publish.yaml (github-release), schema.yaml (diff-schema, validate-schema), spread-docs.yaml (docs-snap-build, docs-spread-tests), spread-manual.yaml (colcon-plugins, extensions, initrd-plugins, kernel-plugins, predicate, remote-build, snap-build, snapd-bases, squid-proxy), spread.yml (build, integration-spread-tests, integration-spread-tests-store)

⚠️ DO NOT MERGE: This PR is intended solely to verify downstream CI workflows.

[DO NOT MERGE] Point starflow workflows to canonical/starflow@work/testing-skill
Disabled non-starflow workflows: spread-manual.yaml, spread-docs.yaml, schema.yaml, release-publish.yaml, publish.yaml, copilot-setup-steps.yaml, copilot-collections-update.yaml, check-renovate.yaml, spread.yml
Copilot AI lite review requested due to automatic review settings October 15, 2026 01:09
@lengau lengau closed this Oct 15, 2026
@lengau
lengau deleted the work/bot/starflow-test-pr-201 branch October 15, 2026 01:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Workflow preservation, QA scope, policy title validation, and TICS trigger coverage must be corrected.

0 open findings

What changed in this PR

This draft PR tests Starflow workflows from work/testing-skill and isolates downstream CI by removing unrelated workflows.

Changes:

  • Updated QA, policy, and TICS workflow references.
  • Removed nine workflows instead of disabling their jobs with if: false.
  • Retained an active QA filter and introduced policy/TICS test inconsistencies.
File Summary
.github/​workflows/​tics.yaml Updated ref, but the generated test branch is not covered by its triggers.
.github/​workflows/​spread.yml Removed instead of preserving and disabling jobs.
.github/​workflows/​spread-manual.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​spread-docs.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​schema.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​release-publish.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​qa.yaml Updated refs, but the filter job remains active despite the description.
.github/​workflows/​publish.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​policy.yaml Updated refs, but the generated title fails the Conventional Commits check.
.github/​workflows/​copilot-setup-steps.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​copilot-collections-update.yaml Removed instead of preserving and disabling jobs.
.github/​workflows/​check-renovate.yaml Removed instead of preserving and disabling jobs.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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