Skip to content

Conversation

@DaleSeo
Copy link
Member

@DaleSeo DaleSeo commented Oct 26, 2025

답안 제출 문제

  • 문제 1
  • 문제 2
  • 문제 3

작성자 체크 리스트

  • Projects의 오른쪽 버튼(▼)을 눌러 확장한 뒤, Week를 현재 주차로 설정해주세요.
  • 문제를 모두 푸시면 프로젝트에서 StatusIn Review로 설정해주세요.
  • 코드 검토자 1분 이상으로부터 승인을 받으셨다면 PR을 병합해주세요.

검토자 체크 리스트

Important

본인 답안 제출 뿐만 아니라 다른 분 PR 하나 이상을 반드시 검토를 해주셔야 합니다!

  • 바로 이전에 올라온 PR에 본인을 코드 리뷰어로 추가해주세요.
  • 본인이 검토해야하는 PR의 답안 코드에 피드백을 주세요.
  • 토요일 전까지 PR을 병합할 수 있도록 승인해주세요.

TonyKim9401
TonyKim9401 previously approved these changes Oct 27, 2025
Copy link
Contributor

@TonyKim9401 TonyKim9401 left a comment

Choose a reason for hiding this comment

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

매번 확인한다는걸 까먹네요..죄송합니다..

@SamTheKorean
Copy link
Contributor

ci설정을 하면 매번 수기로 변경하는 번거로움이 사라질 것 같습니다! 기여감사합니다!

@DaleSeo
Copy link
Member Author

DaleSeo commented Nov 2, 2025

매번 확인한다는걸 까먹네요..죄송합니다..

@TonyKim9401 님, 아직 Draft PR인데 왜 승인하시는 거에요 ㅎㅎ fork 저장소로 부터 만든 PR은 프로젝트 데이터에 접근할 수 없는 문제에 부딪혀서 아직 해결하지 못했습니다. 다음 기수 시작 전까지 완료하는 것이 목표입니다.

@TonyKim9401 TonyKim9401 dismissed their stale review November 2, 2025 16:00

제 실수입니다 철회해두겠습니다.

@DaleSeo
Copy link
Member Author

DaleSeo commented Nov 4, 2025

현재 문제 요약

  • GitHub Actions에서 사용하는 GITHUB_TOKEN을 가지고는 프로젝트 데이터에 접근할 수 없음
  • Secrects에 PAT나 GitHub App 토큰을 저장해두고 불러오면 프로젝트 데이터에 접근할 수 있으나, fork 저장소의 브랜치를 병합하기 위한 PR에서는 보안상 Secrects에 접근할 수 있음
  • pull_request_target 이벤트를 통해 이러한 제한을 우회할 수는 있지만 위험한 선택

@DaleSeo DaleSeo force-pushed the project-validation branch 2 times, most recently from 31a0536 to d9515d1 Compare November 5, 2025 00:28
@DaleSeo
Copy link
Member Author

DaleSeo commented Nov 5, 2025

또 다른 문제들

  • GitHub Actions는 Projects 변경을 직접 감지할 수 없음

@DaleSeo DaleSeo force-pushed the project-validation branch from d9515d1 to 8813c5c Compare November 5, 2025 00:37
@DaleSeo DaleSeo force-pushed the project-validation branch from 8813c5c to 39f012f Compare November 5, 2025 00:44
@DaleSeo
Copy link
Member Author

DaleSeo commented Nov 5, 2025

/check-week

@DaleSeo DaleSeo force-pushed the project-validation branch from 223f32e to 045774d Compare November 5, 2025 00:52
PR comments use GitHub Issues API internally, so issues:write permission is required.
- Suppress stderr output for gh commands
- Continue workflow even if comment posting fails
- Add clearer error messages
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

⚠️ Week 설정이 누락되었습니다

프로젝트에서 Week를 설정해주세요!

설정 방법

  1. PR 우측의 Projects 섹션에서 리트코드 스터디 옆 드롭다운(▼) 클릭
  2. 현재 주차를 선택해주세요 (예: Week 14(current) 또는 Week 14)

📚 자세한 가이드 보기


🤖 이 댓글은 GitHub App을 통해 자동으로 작성되었습니다.

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.

3 participants