오픈소스 첫 PR을 해보다.
문제 정의 : 먼저 doc에 blob.bytes()가 포함되지 않았다는 이슈가 존재하였습니다.
이 문제는 doc 관련 문제여서 일단 차근차근 해보자라는 생각에 시작하게 되었습니다.
먼저 Blob이 대용량 바이너리 데이터를 다룰 때 필요한 객체에서 처음에는 bytes()로 반환하여 올렸습니다.
그리고 PR을 처음 보내봐서 test.txt라는 파일도 잘 못 보내게 되었습니다.
이후, 두 분께서 피드백을 해주셨습니다.
일단 test.txt를 빼고 다시 PR을 보내고, 주석을 다시 달아달라고 첫 번째 분께서 말씀해주셨습니다.
그리고 두 번째 분께서 blob.bytes()의 반환값을 다시 아리켜주셨고, 추가적으로 commit message guide를 맞춰서 commit을 보내달라고 하셨습니다.
그래서 이 분들의 말씀을 듣고 test.txt파일을 삭제하고,
Promise<Uint8Array> 형식으로 반환값을 바꾸어 제출하였습니다.
이후
squashing을 통해 2개의 커밋을 하나로 합쳐달라고 하셔서 해결하였습니다.
이후 버전을 넣어주라고 하셔서 버전을 넣고 처음이라 서투르다고 양해의 말씀을 드리고, 더 필요하신게 있으시면 이야기 해주시라고 말했습니다.
그 다음 커밋 2개 중 마지막 커밋이 문제가 발생하였다고 해서 이 커밋을 삭제한 뒤 한 개의 커밋만 남기도록 하였습니다.
그 이후 멘토님께서 알파벳 순으로 변경해달라고 하셔서 수정하고,
**targos** 분께서 변경할 때 생긴 공백을 삭제하고 버전을 하나 더 넣어달라고 요청하여서 수정하였습니다.
이후 처음으로 기여자가 되보았습니다 ! 😎
이번 PR을 해보면서 PR의 전체 구조(이슈 할당 > 작업 > PR > 머지))는 해본 것 같아서 스스로 자랑스러웠습니다. node.js를 잘 몰랐지만 스스로 찾아가고, 공부하면서 하나의 오픈소스 이슈에 대해서 기여할 수 있다는 것에 기뻤습니다. 또한 모르는 사람이지만 오픈소스 하나로 묶여서 서로를 도와주고 같이 문제를 해결하는 오픈소스 생태계에 매력을 느꼈습니다. 그래서 더 열심히 오픈소스 생태게에 빠져들어 열심히 기여할 생각입니다.
PR 방법은 다음 이슈를 해결하면서 천천히 설명드리겠습니다.
