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

config: release note #64

Merged
merged 4 commits into from
Apr 16, 2024
Merged

config: release note #64

merged 4 commits into from
Apr 16, 2024

Conversation

Hanna922
Copy link
Member

1️⃣ 어떤 작업을 했나요? (Summary)

2️⃣ 알아두시면 좋아요!

여기 Github Action 사용하려면 개인 Github Token이 있어야 해서, 제 걸로 일단 등록해둘게용
Ref: https://yoonah-dev.oopy.io/0b8bb991-b719-4c5c-b2b2-39a55305a3ed

4️⃣ 체크리스트 (Checklist)

  • main 브랜치의 최신 코드를 pull 받았나요?

@Hanna922 Hanna922 self-assigned this Apr 16, 2024
@Hanna922 Hanna922 requested a review from nijuy as a code owner April 16, 2024 01:41
Copy link
Collaborator

@nijuy nijuy left a comment

Choose a reason for hiding this comment

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

등교 이슈로 확인이 늦었습니다 반성중 (⊙x⊙;)

Comment on lines 6 to 15
- title: '🆕 새로운 기능이 추가되었어요!'
label: '✨ Feature'
- title: '🐞 자잘한 버그를 수정했습니다.'
label: '🐞 Bugfix'
- title: '🫶🏻 앱 사용성 개선에 힘썼습니다.'
label: '🫶🏻 Improvement'
- title: '🛠️ 더 나은 코드를 위해 노력하고 있습니다.'
labels:
- '🔨 Refactor'
- '⚙️ Setting'
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • PR의 label 기준으로 나뉘는 거 같은데, 현재 사용 중인 라벨이랑 달라서 다 ETC 로 들어갈 거 같습니다.

  • "앱 사용성 개선"과 "더 나은 코드" 부분이 어떤 차이가 있을지 생각해봤는데, 일반 서비스에서는 사용자에게 직접적으로 와닿는 UX적인 개선과 (그렇지 않은) 코드 개선으로 나눌 수 있을 거 같아요

    근데 디자인 시스템에서는 얘네를 나눠야 할지 잘 모르겠읍니다
    "사용성 개선" 하나로 퉁치면 어떨까 싶은데 어떠세여

Suggested change
- title: '🆕 새로운 기능이 추가되었어요!'
label: '✨ Feature'
- title: '🐞 자잘한 버그를 수정했습니다.'
label: '🐞 Bugfix'
- title: '🫶🏻 앱 사용성 개선에 힘썼습니다.'
label: '🫶🏻 Improvement'
- title: '🛠️ 더 나은 코드를 위해 노력하고 있습니다.'
labels:
- '🔨 Refactor'
- '⚙️ Setting'
- title: '🆕 새로운 기능이 추가되었어요!'
label: 'feat'
- title: '🐞 자잘한 버그를 수정했습니다.'
labels:
- 'bug'
- 'fix'
- title: '🫶🏻 사용성 개선에 힘썼습니다.'
labels:
- 'refactor'
- 'docs'

Copy link
Member Author

@Hanna922 Hanna922 Apr 16, 2024

Choose a reason for hiding this comment

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

refactor는 YDS 내부 코드 개선에 대한 작업이 더 많은 것 같아서 유저의 사용성 개선에는 포함이 안될 것 같은 느낌이 드네여
pnpm으로 바꾼다거나, 버전을 올린다거나.. 등등
사용성 개선을 docs로 두고, 더 나은 코드를 위해 노력을 refactor로 두는 거 어떠신가요?

Copy link
Collaborator

@nijuy nijuy Apr 16, 2024

Choose a reason for hiding this comment

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

좋네요 !

slack-imgs

Copy link
Member Author

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ표정 ㄱㅇㅇ

Copy link
Member Author

Choose a reason for hiding this comment

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

생각해보니까 refactor랑 chore도 같이 사용하는 것 같아서 같이 넣었슴다

with:
config-name: release.yml
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

제 걸로 일단 등록해둘게용

개인 토큰을 조직에서 쓰려면 따로 승인해야 하는 거 같아요 아마도. 한 34%의 확률로............. 저도 확신은 못하지만............... (문서, 문서 2)

Copy link
Member Author

Choose a reason for hiding this comment

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

오음 이거 일단 YDS repo 내 token으로 등록할 건데, 테스트 해보고 안되면 레포 키로 등록할게용

@Hanna922 Hanna922 merged commit 3863b16 into develop Apr 16, 2024
@Hanna922 Hanna922 deleted the config/release-note branch April 16, 2024 13:00
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.

feat: 릴리즈 노트 자동화
2 participants