Skip to content

feat: strip_components 설정 추가#2175

Merged
Soundbar91 merged 1 commit intodevelopfrom
fix/cd-workflows
Mar 13, 2026
Merged

feat: strip_components 설정 추가#2175
Soundbar91 merged 1 commit intodevelopfrom
fix/cd-workflows

Conversation

@Soundbar91
Copy link
Collaborator

@Soundbar91 Soundbar91 commented Mar 13, 2026

🔍 개요


🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

Summary by CodeRabbit

  • Chores
    • Updated deployment process configuration to improve file transfer handling during releases to development and production servers.

@Soundbar91 Soundbar91 self-assigned this Mar 13, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 13, 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: bc01a0c4-68c4-45e8-973d-d63937b74781

📥 Commits

Reviewing files that changed from the base of the PR and between d501df8 and 8fc9284.

📒 Files selected for processing (2)
  • .github/workflows/backend-cd-develop.yml
  • .github/workflows/backend-cd-main.yml

📝 Walkthrough

Walkthrough

Two GitHub Actions workflow files were updated to add the strip_components: 2 parameter to the SCP JAR file transfer steps. This modifies how source paths are stripped during extraction on target servers in both develop and main deployment pipelines.

Changes

Cohort / File(s) Summary
SCP Action Configuration
.github/workflows/backend-cd-develop.yml, .github/workflows/backend-cd-main.yml
Added strip_components: 2 parameter to SCP JAR file transfer steps to strip the first two path components during extraction on target servers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Strip and hop, the components flee!
Two workflows now agree,
JAR files dance to paths anew,
Development and main both true!
With strip_components: 2 so spry,
Deployments reach their destined sky! 🚀

🚥 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 is in Korean and refers to 'strip_components 설정 추가' (adding strip_components configuration), which directly matches the changes made to both backend-cd-develop.yml and backend-cd-main.yml files.
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
  • Commit unit tests in branch fix/cd-workflows
📝 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 use OpenGrep to find security vulnerabilities and bugs across 17+ programming languages.

OpenGrep is compatible with Semgrep configurations. Add an opengrep.yml or semgrep.yml configuration file to your project to enable OpenGrep analysis.

@github-actions github-actions bot added the 버그 정상적으로 동작하지 않는 문제상황입니다. label Mar 13, 2026
Copy link

@ff1451 ff1451 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

@github-actions
Copy link

Unit Test Results

690 tests   687 ✔️  1m 22s ⏱️
169 suites      3 💤
169 files        0

Results for commit 8fc9284.

@Soundbar91 Soundbar91 merged commit dda0d2e into develop Mar 13, 2026
9 checks passed
@Soundbar91 Soundbar91 deleted the fix/cd-workflows branch March 13, 2026 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 정상적으로 동작하지 않는 문제상황입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants