Skip to content

ci(test): Also run on the dev branch#88

Merged
SteakFisher merged 1 commit into
mainfrom
SteakFisher-patch-2
Jun 13, 2026
Merged

ci(test): Also run on the dev branch#88
SteakFisher merged 1 commit into
mainfrom
SteakFisher-patch-2

Conversation

@SteakFisher

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Jayadeep Bejoy <88953813+SteakFisher@users.noreply.github.com>
@SteakFisher SteakFisher merged commit e83ee9b into main Jun 13, 2026
2 checks passed
@SteakFisher SteakFisher deleted the SteakFisher-patch-2 branch June 13, 2026 19:40
@greptile-apps

greptile-apps Bot commented Jun 13, 2026

Copy link
Copy Markdown

Greptile Summary

This PR fixes the CI test.yml workflow by correcting the pull_request branch target from develop to dev, matching the actual branch name in the repository (origin/dev). The old value was referencing a branch that doesn't exist, meaning tests were never triggered for PRs targeting dev.

  • Renames the develop branch entry to dev in the on.pull_request.branches filter so the "Run Tests" workflow correctly fires on PRs opened against the dev branch.

Confidence Score: 5/5

Safe to merge — a one-line correction that makes the CI trigger match the real branch name.

The only change is renaming develop to dev in the branch filter. The origin/dev branch already exists in the repo, so this fix ensures tests actually run on PRs targeting that branch, as originally intended.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/test.yml Renames the develop branch trigger to dev, aligning the PR target filter with the actual remote branch that exists in the repo (origin/dev).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR Opened / Synchronized / Reopened] --> B{Target Branch?}
    B -->|main| C[Run Tests ✅]
    B -->|dev ← renamed from 'develop'| C
    B -->|other| D[Skip Tests]
    C --> E[Type check with tsc]
Loading

Reviews (1): Last reviewed commit: "ci(test): Also run on the dev branch" | 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.

1 participant