Nomad flutter study의 리뷰를 위한 repository 입니다.
1. 현재 repository에서 fork 버튼을 눌러 본인 github 계정 소유의 repository 를 만듭니다.
cd flutter-study
git remote add review-origin https://github.com/{githubId}/nlutter-study-2nd
git remote add review-upstream https://github.com/nlutter-study/nlutter-study-2nd# cd flutter-study
git checkout -b day12-movieflix # 과제명예시flutter create project와과제구현부가 하나의 commit 으로 되어있는 경우 rebase 를 통해 두 개의 commit 으로 분리해 주시는 것이 리뷰에 용이합니다. 저같은 경우lib/*를 맨 마지막에 commit하고 그 나머지를 미리 commit해두는 편입니다.flutter create project와과제구현부commit 분리된 결과물의 예시
$ git log --name-only
commit 9450412149aba694eda2111b87f0cd9780c94f7a
Author: devgony <[email protected]>
Date: Tue Oct 24 23:57:30 2023 +0900
Day14-Movieflix!
movieflix/lib/main.dart
movieflix/lib/models/movie_detail_model.dart
#...
commit 2eb60d867e493e5307ab968c307d844b8952347c
Author: devgony <[email protected]>
Date: Tue Oct 24 16:36:31 2023 +0900
build: create movieflix project
movieflix/android/.gitignore
movieflix/ios/Flutter/AppFrameworkInfo.plist
movieflix/linux/flutter/CMakeLists.txt
#...- githubId 가 존재하지 않는경우 henry 에게 문의 부탁 드립니다.
# cd flutter-study
git fetch review-upstream {githubId}
git rebase review-upstream/{githubId}5. review-origin 에 push 하고 review-upstream 접속하여 PR을 생성합니다.
- 리뷰 대상이 아닌 파일과 commit이 섞여 있는 경우 어떤 Commit부터 리뷰의 대상인지 설명을 작성 해주시면 좋습니다.
- 내용에는 간단하게 해당 PR에 대한 설명과 구현할 목표(Todo)를 작성해주시면 좋습니다.
- 하나의 디렉토리 내에 각 과제 프로젝트가 폴더 로 분리되어있는 monorepo 방식으로 개발 해주시면 추후 리뷰의 편의성을 높일 수 있습니다.
# cd flutter-study
git push review-origin day12-movieflix- review-origin에 push 직후 review-upstream 에 접속하면 아래와 같은 PR 생성 버튼이 보입니다.
base repository 는 반드시 본인 githubId로 해주셔야 합니다.
React
Typescript
Django
Laravel
Facebook
Microsoft
Google
Alibaba
D3
Tencent