Skip to content

Conversation

@Hexeong
Copy link
Contributor

@Hexeong Hexeong commented Nov 15, 2025

관련 이슈

작업 내용

  1. GHCR의 이미지를 pull하기 위한 Github App으로 토큰을 발행하는 과정에서 구 버전의 문법을 사용해서 오류가 났습니다.
  2. 해당 오류를 수정했습니다.

특이 사항

리뷰 요구사항 (선택)

  • 해당 pr이 제대로 적용되는지 보기 위해서는 먼저 머지가 되어야 합니다.. 이 부분 양해 부탁드립니다..

@Hexeong Hexeong self-assigned this Nov 15, 2025
@Hexeong Hexeong added the 버그 Something isn't working label Nov 15, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 15, 2025

Walkthrough

  1. dev-cd.yml: 탭 내부에서 "Create installation token" 스텝의 입력 키를 permissions: "packages:read"에서 permission-packages: "read"로 변경했고, 추가로 permission-contents: "read" 입력을 추가했습니다.
  2. prod-cd.yml: 탭 내부에서 dev-cd.yml과 동일하게 permissions: "packages:read"permission-packages: "read"로 교체하고 permission-contents: "read"를 추가했습니다.
  3. 두 워크플로우 모두 actions/create-github-app-token@v2 액션의 입력명만 변경/추가되었고, 다른 제어 흐름이나 스텝에는 변경이 없습니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • 동일한 패턴의 반복적 변경이라 복잡도 낮음.
  • 입력 키 네이밍 변경 및 추가만 있어 로직 검증 부담 적음.

검토 시 주의할 파일:

  • .github/workflows/dev-cd.yml
  • .github/workflows/prod-cd.yml

Suggested reviewers

  • wibaek
  • whqtker
  • lsy1307

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed PR 설명이 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 내용, 특이사항이 모두 포함되어 있습니다.
Linked Issues check ✅ Passed PR의 코드 변경사항이 #564 이슈의 목표를 충족하고 있습니다. CD 프로세스의 GitHub App 권한 명시 오류가 구 문법에서 신 문법으로 수정되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #564 이슈의 범위 내에 있으며, GitHub App 토큰 권한 설정 수정에만 집중하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly and specifically addresses the main change: fixing GitHub App token permission syntax errors in the CD workflow files (dev-cd.yml and prod-cd.yml).
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@Hexeong Hexeong changed the title Fix/564 GitHub app token permission fix: GitHub app token permission 문제 해결 Nov 15, 2025
@Hexeong Hexeong merged commit 26490f5 into solid-connection:develop Nov 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: CD process에서 Github App에 대한 권한 명시 오류 수정

2 participants