chore: stop running CI twice per change - #19
Merged
Conversation
pull_request 와 push 를 둘 다 develop 에 걸어두는 바람에 PR 에서 한 번, 머지 커밋에서 또 한 번 돌았다. 같은 코드를 두 번 검사하면서 러너 시간을 쓰고 알림도 두 배로 나갔다. Android 빌드가 10분대라 체감이 컸다. develop 은 PR 을 통과한 커밋만 들어오므로 머지 후 재검사는 얻는 게 없다. push 트리거는 main 에만 남긴다. 릴리스 지점이라 한 번 더 볼 값어치가 있다.
알림과 체크 목록에 뜨는 이름이라 영어로 통일. 파일 안 주석은 저장소의 다른 문서와 맞춰 한국어로 둔다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
develop에 pull_request랑 push가 둘 다 걸려 있어서 PR에서 한 번, 머지 커밋에서 또 한 번 돌았음. push는 main만 남김.