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

fix(Geun-Oh/internal_developer_platform_vs_internal_developer_portal): typo #8

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

jjangga0214
Copy link
Member

@Geun-Oh 첫번째 글 정말 수고하셨습니다 :)

만약 이 PR 에 대해 다른 의견이 있으시면 말씀주세요!

@jjangga0214 jjangga0214 requested a review from Geun-Oh as a code owner February 24, 2024 12:55
Copy link
Contributor

✅ 사이드 이펙트가 감지되지 않았습니다.
(PR 에 포함되지 않은 파일들의 렌더링에 영향이 없습니다.)

@Geun-Oh
Copy link
Member

Geun-Oh commented Feb 24, 2024

아아 감사합니다!! 미처 진행하지 못한 검수 사항이 있었군요.
다음에는 더 꼼꼼하게 확인해보겠습니다 :)

@jjangga0214
Copy link
Member Author

jjangga0214 commented Feb 24, 2024

p.s. #5 에서 main 에 대한 커밋이 덮어씌워져서 squash 로 정책을 바꾸었습니다 :)

아 또 여담으로, PR merge 시에는 commit message 에 [ci skip]/[skip ci] 가 포함되지 않도록 하시면 좋습니다!

@Geun-Oh
Copy link
Member

Geun-Oh commented Feb 24, 2024

p.s. #5 에서 main 에 대한 커밋이 덮어씌워져서 squash 로 정책을 바꾸었습니다 :)

아 또 여담으로, PR merge 시에는 commit message 에 [ci skip]/[skip ci] 가 포함되지 않도록 하시면 좋습니다!

아 제가 squash 하는 과정에서 base commit을 잘못 기입한 것 같습니다. 확인했습니다!

@jjangga0214 jjangga0214 merged commit 9168b7f into main Feb 25, 2024
3 checks passed
@jjangga0214 jjangga0214 deleted the fix/typo branch February 25, 2024 03:41
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