Skip to content

fix: github workflow syntax error#54

Merged
ga111o merged 1 commit intomainfrom
fix/workflow-syntax-err
Apr 9, 2026
Merged

fix: github workflow syntax error#54
ga111o merged 1 commit intomainfrom
fix/workflow-syntax-err

Conversation

@ga111o
Copy link
Copy Markdown
Member

@ga111o ga111o commented Apr 9, 2026

Summary

pr시 빌드하는 workflow파일에 단순한 문법 오류가 있었습니다.

Type of Change

  • Bug fix
  • New feature
  • Refactoring
  • Docs / Config
  • CI/CD

Testing

  • Tested locally
  • No regression in existing functionality

Checklist

  • Reviewers assigned
  • Related issue linked

Summary by CodeRabbit

  • Chores
    • Corrected GitHub Actions workflow configuration syntax to ensure proper script termination.

@ga111o ga111o requested review from kwonkwonn and ukleon123 April 9, 2026 04:48
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 233dcbb2-bdaf-4d29-b4a9-9c35aa836504

📥 Commits

Reviewing files that changed from the base of the PR and between 01a7311 and 8dfdd6f.

📒 Files selected for processing (1)
  • .github/workflows/pr-build-check.yaml
💤 Files with no reviewable changes (1)
  • .github/workflows/pr-build-check.yaml

📝 Walkthrough

Walkthrough

Removed an extraneous closing }); from a github-script action in the PR build check workflow, correcting JavaScript statement termination and eliminating a trailing newline issue.

Changes

Cohort / File(s) Summary
GitHub Workflows
.github/workflows/pr-build-check.yaml
Removed extra closing bracket (});) from inline github-script block and fixed file ending state for proper script termination.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A bracket too many caused quite the fuss,
In workflow scripts, code flows were clogged up!
With one swift delete, the syntax runs clean,
GitHub actions now execute supreme! 🚀
The rabbit rejoices—control flow's so neat!

🚥 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 'fix: github workflow syntax error' directly and clearly describes the main change—removing a syntax error from a GitHub workflow file.
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
  • Commit unit tests in branch fix/workflow-syntax-err

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.

@ga111o ga111o merged commit f8ff72d into main Apr 9, 2026
4 checks passed
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