ci: test pr-cancel 0xbigapple

[317787106]

What does this PR do?

Why are these changes required?

This PR has been tested by:

• Unit Tests
• Manual Testing

Follow up

Extra details

---

Summary by cubic

Update the PR cancel GitHub Actions workflow to run only for unmerged PRs. Adds an if: github.event.pull_request.merged == false guard to the "Cancel In-Progress Workflows" job to prevent unnecessary runs on merged PRs.

^{Written for commit 1d968f6. Summary will update on new commits.}

Summary by CodeRabbit

• Chores
  • Improved pull request workflow handling to prevent unnecessary operations on merged pull requests.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c2ce706d-4952-4e7f-a9ce-1f13e71a5912

📥 Commits

Reviewing files that changed from the base of the PR and between 081e02b and 1d968f6.

📒 Files selected for processing (1)

• .github/workflows/pr-cancel.yml

---

📝 Walkthrough

Walkthrough

A conditional guard was added to the cancel job in the GitHub Actions workflow to prevent execution when a pull request is merged, ensuring cancellation logic only applies to active, unmerged PRs.

Changes

Cohort / File(s)	Summary
Workflow Guard Condition .github/workflows/pr-cancel.yml	Added job-level condition if: github.event.pull_request.merged == false to the cancel job to skip cancellation logic for merged pull requests.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• #56: Modifies the same pr-cancel.yml workflow file and adjusts cancellation logic, directly related to this PR's workflow improvements.

Poem

🐰 A merged PR deserves its rest,
No cancellation serves it best,
With one small if the task is done,
Now workflows know when they've won! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'ci: test pr-cancel 0xbigapple' is vague and does not clearly describe the main change—it uses generic terms like 'test' and includes what appears to be a username, making it unclear what the actual PR accomplishes.	Revise the title to clearly describe the main change, such as 'ci: add condition to skip pr-cancel workflow for merged PRs' or similar, avoiding vague terms and unnecessary identifiers.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.

Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present.

[cubic-dev-ai]

No issues found across 1 file