Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: workflow 테스트 #401

Merged
merged 4 commits into from
Jan 21, 2025
Merged

feat: workflow 테스트 #401

merged 4 commits into from
Jan 21, 2025

Conversation

jeamxn
Copy link
Member

@jeamxn jeamxn commented Jan 21, 2025

상태

  • 준비됨
  • 개발중
  • 일시중지

설명

기타

리뷰 원칙

P1: 꼭 반영해주세요 (Request changes)

리뷰어는 PR의 내용이 서비스에 중대한 오류를 발생할 수 있는 가능성을 잠재하고 있는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우, P1 태그를 통해 리뷰 요청자에게 수정을 요청합니다. 리뷰 요청자는 p1 태그에 대해 리뷰어의 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다.

P2: 적극적으로 고려해주세요 (Request changes)

작성자는 P2에 대해 수용하거나 만약 수용할 수 없는 상황이라면 적합한 의견을 들어 토론할 것을 권장합니다.

P3: 웬만하면 반영해 주세요 (Comment)

작성자는 P3에 대해 수용하거나 만약 수용할 수 없는 상황이라면 반영할 수 없는 이유를 들어 설명하거나 다음에 반영할 계획을 명시적으로(JIRA 티켓 등으로) 표현할 것을 권장합니다. Request changes 가 아닌 Comment 와 함께 사용됩니다.

P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)

작성자는 P4에 대해서는 아무런 의견을 달지 않고 무시해도 괜찮습니다. 해당 의견을 반영하는 게 좋을지 고민해 보는 정도면 충분합니다.

P5: 그냥 사소한 의견입니다 (Approve)

작성자는 P5에 대해 아무런 의견을 달지 않고 무시해도 괜찮습니다.

Copy link

vercel bot commented Jan 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dipull ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 21, 2025 8:46am

@jeamxn jeamxn changed the title [FEAT] workflow 테스트 feat: workflow 테스트 Jan 21, 2025
@jeamxn jeamxn self-assigned this Jan 21, 2025
@jeamxn jeamxn requested review from yeonfish6040 and sspzoa January 21, 2025 08:48
Copy link
Contributor

@yeonfish6040 yeonfish6040 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에서 확일할 필요가 있을까요?

@jeamxn jeamxn merged commit c2f9b2e into main Jan 21, 2025
4 checks passed
@jeamxn jeamxn deleted the feat/test-workflow branch January 21, 2025 09:50
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