Skip to content

Fix malformed Resolution/Done condition in resolution notifier workflow#4804

Open
Deathbot545 wants to merge 1 commit intowso2:mainfrom
Deathbot545:fix/workflow-resolution-label-condition
Open

Fix malformed Resolution/Done condition in resolution notifier workflow#4804
Deathbot545 wants to merge 1 commit intowso2:mainfrom
Deathbot545:fix/workflow-resolution-label-condition

Conversation

@Deathbot545
Copy link
Copy Markdown

@Deathbot545 Deathbot545 commented Mar 14, 2026

  • Purpose
    Fixed a minor syntax error in the issue-clousre workflow condition
  • Issue
    N/A no issues Linked

Summary by CodeRabbit

  • Chores
    • Fixed a syntax error in the automated workflow configuration to ensure proper label processing.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 14, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: afc43c00-453e-4d42-b225-17d4c1b3ee7b

📥 Commits

Reviewing files that changed from the base of the PR and between c16a70e and 94d34cf.

📒 Files selected for processing (1)
  • .github/workflows/resolution_label_notifier.yml

Walkthrough

A syntax error in a GitHub Actions workflow file was corrected. The label condition check was missing a closing single quote, changing 'Resolution/Done) to 'Resolution/Done'). This resolves a YAML syntax issue preventing the workflow from executing properly.

Changes

Cohort / File(s) Summary
Workflow Syntax Fix
.github/workflows/resolution_label_notifier.yml
Corrected missing closing quote in the label condition check within the workflow trigger logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A tiny quote was lost and alone,
On line where the resolution was known.
One swift keystroke, a hop and a bound,
The workflow now works—syntax is sound! 🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing a syntax error in the Resolution/Done condition within the resolution notifier workflow, which is the sole modification in this PR.
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
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Tip

CodeRabbit can approve the review once all CodeRabbit's comments are resolved.

Enable the reviews.request_changes_workflow setting to automatically approve the review once all CodeRabbit's comments are resolved.

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