Skip to content

fix(ci): make cli_tests.yml a valid workflow#900

Open
aiSynergy37 wants to merge 1 commit intoHelixDB:mainfrom
aiSynergy37:fix/cli-tests-workflow-valid-trigger
Open

fix(ci): make cli_tests.yml a valid workflow#900
aiSynergy37 wants to merge 1 commit intoHelixDB:mainfrom
aiSynergy37:fix/cli-tests-workflow-valid-trigger

Conversation

@aiSynergy37
Copy link
Copy Markdown

@aiSynergy37 aiSynergy37 commented Apr 9, 2026

Summary

  • replace the invalid/commented cli_tests.yml workflow with a valid workflow definition
  • use workflow_dispatch + a placeholder job so GitHub no longer reports parser failures like No event triggers defined in on

Why

PRs were receiving workflow failure notifications even when no jobs could run because the workflow file had no valid on section.

Note

This intentionally keeps CLI tests disabled for automatic PR triggers; it only fixes workflow validity/noise.

Greptile Summary

This PR fixes a GitHub Actions parser failure caused by cli_tests.yml being entirely commented out, leaving no valid on: trigger. The file is replaced with a minimal valid workflow (workflow_dispatch + a no-op placeholder job) so GitHub no longer reports parse errors on PRs, while keeping CLI tests disabled for automatic triggers.

Important Files Changed

Filename Overview
.github/workflows/cli_tests.yml Replaces an entirely commented-out (invalid) workflow file with a minimal valid workflow using workflow_dispatch and a placeholder no-op job to silence GitHub Actions parser errors.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[GitHub detects cli_tests.yml] --> B{Valid on: trigger?}
    B -- "Before (all commented out)" --> C[Parser Error: No event triggers defined]
    C --> D[Workflow failure notification on PRs]
    B -- "After (workflow_dispatch)" --> E[Workflow is valid]
    E --> F{Triggered manually?}
    F -- Yes --> G[Run noop placeholder job]
    G --> H[echo message and exit]
    F -- No --> I[Workflow not triggered on PRs]
    I --> J[No spurious failure notifications]
Loading

Reviews (1): Last reviewed commit: "fix(ci): make cli_tests workflow valid t..." | Re-trigger Greptile

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