Pull Request와 Code review 진행 상황
[1] Pull Request 진행 상황(Merge된 PR 기준)
-Comment 수정에 관한 Pull Request
reorder-comment-Sort-alphabetically
[UPDATE] fix comment of method_sorter.py
Fix a typing error of comments in pfb1-10
Fix a typing error of comments in pfb11-18
Fix errors in comments in mpfb 1-9
-Comment 추가에 관한 Pull Request
Add coment 1
[UPDATE]Add comment
-README 수정에 관한 Pull Request
[FIX]fix a typing error
Fast-forward project
-File Name 수정에 관한 Pull Request
[UPDATE]Fix file name
[UPDATE]Changed all folder name Slides to 00 - Slides
[Update] Changed folder name ‘Slides’ to ‘00 - Slides’ in python-for-beginners
Revert “[UPDATE] Changed all folder name Slides to 00 - Slides”
[UPDATE]Changed folder name “Slides” to “00 - Slides” in better-more-python-for-beginners
-개별 py파일 내용 수정 및 추가에 관한 Pull Request
Fix code challenge solution.py
Fix method_sorter.py in more-python/chapter 2
[ADD]add simple example using decorator
Added more details to decorators_simple.py
-Unifying Indentation에 관한 Pull Request
Unifying indentation: python-for-beginners/chapter 2-4
Unifying indentation: python-for-beginners/chapter 5-7
Unifying indentation: python-for-beginners/chapter 8-10
Unifying indentation: python-for-beginners/chapter 11-13
Unifying indentation: python-for-beginners/chapter 14-16
Unifying indentation: python-for-beginners/chapter 17-18
Unifying indentation: more-python-for-beginners/chapter 1-3
Unifying indentation: more-python-for-beginners/chapter 4-6
Unifying indentation: more-python-for-beginners/chapter 7-9
Unifying indentation: even-more-python-for-beginners-… chapter 1-5
Unifying indentation: even-more-python-for-beginners/chapter 6-10
Unifying indentation: even-more-python-for-beginners-… chapter 11-15
Indent unify of python-for-beginners
Indent unify of more-python-for-beginners
Indent unify of even-more-python-for-beginners-data-tools
[2] Code review 진행 상황
c9-python-getting-started는 파이썬에 관한 내용을 알려주는 가이드북이기 때문에, 대부분의 Code Review들은 수정 및 추가된 code와 comment의 문법적 오류와 가독성을 개선하는 방향으로 이루어졌다. 13주차 Code Review 강의 내 Rule of Code Review의 내용에서 컨벤션(코드스타일) 지적은 피하자는 부분이 있지만, 본 프로젝트는 가이드북 프로젝트이고 PEP8도 소개하고 있기 때문에 컨벤션 지적도 함께 이루어졌다.
-예시
- Comment 추가에 관한 Pull Request
- 가독성 개선에 관한 Code Review
- Commit 1차 수정
- Convention 지적에 관한 Code Review
- Commit 2차 수정
위와 같은 방식으로 Code Review와 수정이 이루어졌다.