Repository 선정
Python & JAVA Solutions for Leetcode
=> 주로 code snippet을 제작해둔 Repository를 후보로 생각하였으나, code interview의 중요성을 여러 과목에서 접하였습니다. 이를 바탕으로 Repository를 search하고 선정하였습니다.
ISSUE
분석
Python & JAVA Solutions for Leetcode(ISSUE #3)
Code 내에 주석으로 문제 번호 혹은 문제의 링크가 있길 원함
Python & JAVA Solutions for Leetcode(ISSUE #71)
특정 문제에 대한 해답을 원함
제기
Python & JAVA Solutions for Leetcode(ISSUE #73)
이미 존재하는 해설 중 Test case를 포함한 경우.
오답 코드를 가지고 있는 경우를 확인.
Repository Owner의 코드 였기 때문에 이에 대한 문의를 진행한 후
해당 ISSUE와 연결되는 PR을 생성하여 Close하였음.
PR
분석
Python & JAVA Solutions for Leetcode(PR #8)
code style :
1. Pylint(PEP8)를 적용
2. 가독성을 위해 주석을 작성
Readme 작성 :
1. 해결한 문제는 Readme에 추가
Python & JAVA Solutions for Leetcode(PR #50)
File 네이밍
1. [문제번호][문제이름].[확장자] 형태
2. 문제이름의 공백은 “_”로 대체
Python & JAVA Solutions for Leetcode(PR #36)
code 스타일 :
1. 의미있는 변수 네이밍
2. 파일 마지막 개행 추가
생성 & Merge
Python & JAVA Solutions for Leetcode(PR #60)
새로운 문제 풀이를 커밋(Q2413, Q2409, Q1909)
Python & JAVA Solutions for Leetcode(PR #61)
새로운 문제 풀이를 커밋(Q2420)
이전 PR#60에서 사용했던 Repository를 그대로 사용하였더니,
Log 관리가 어려웠음.
이에 대한 Owner의 피드백이 있어 (fetch 이후 rebase) 이를 반영 하여 추후 진행
Python & JAVA Solutions for Leetcode(PR #63)
새로운 문제 풀이를 커밋(Q2429_Minimize_XOR)
Python & JAVA Solutions for Leetcode(PR #68)
새로운 문제 풀이를 커밋(Q523_Continuous_Subarray_Sum)
문제점 :
- code 스타일 중 하나인 파일 마지막에 개행을 추가하는 것,
- ReadMe 변경을 잘 못 한 것
Python & JAVA Solutions for Leetcode(PR #76)
ISSUE #73을 Close하는 PR
해당 code의 TestCase를 변경하거나,
__main__으로 시작되는 코드 블럭 제거
느낀 점
- 팀원들과의 협업을 통해 해당 Repository의 컨벤션을 분석하고 관습등을 찾는 것이 흥미로웠습니다.
- 본인이 소유하고 있지 않은 Repository에 기여를 하는 경험이 처음이어서 걱정을 많이 했지만 방향성을 잡고, 컨택을 시도하여 성공적으로 수행한 것 같아 다행입니다.
- 활발하게 활동하는 Repository를 선정할 수 있어 다행이었습니다.
