[2020년 12월 11일 기술블로그에서 작성]
💚이전글 읽기💚
1. passcode 화면에서 뒤로가기 누르면 앱 닫기 (Issue #7343)
2. passcode 화면에서 뒤로가기 누르면 앱 닫도록 코드 구현 후 PR
3. passcode 화면에서 뒤로가기 이벤트 PR -> Merge
Nextcloud/android Repository에서도 이 글을 읽었던 것 같아서 확인해보니 동일한 설명이 android에도 있었다. 저번 코드 기여했을 때 copyright을 작성하지 않고 merge가 되었는데,,늦었지만 나도 copyright을 추가하면 좋겠다고 생각했다.
여기에 몇가지 고민사항이 있었다.
첫번째로 나는 하나의 클래스나 하나의 함수를 새로 구현한 것이 아닌 기존에 있던 함수의 일부로 새 기능을 추가 구현한 것 인데, 이것도 copyright 추가를 할 수 있을까?🤔
두전째는 몇가지 파일을 열어서 확인해보니 member거나 collaborator만 copyright을 적는 것 같았다.😥
세번째로 나는 이미 merge가 되었는 데, copyright comment만 추가하는 걸 멤버개발자들이 merge해줄지 걱정되었다.😩
그래도 시도라도 해보자! 라는 마음에 Nextcloud/android 멤버개발자인 tobias에게 메일을 보냈다.
거절 답변 쪽으로 올거라 생각하고 있었다. 혹시 늦어서 copyright을 안 받아주는 거면,, 코드 기여하면서 바로 같이 안 올린 내 잘못이니까,,, 어쩔 수 없지😭😭 다음날 tobias에게 답장이 왔는 데, copyright을 올려도 된다는 답변이었다!!!!
답장을 받자마자 copyright을 추가해서 PR을 날렸고, 다른 멤버 개발자 Andy가 merge를 해줬다. #7683
시험기간이라서 요새 활동을 못 했는데, Andy가 welcome back이라고,,했다.🤣🤣
아직 시험이 안끝나서 활동을 못 합니다,,, 끝나고 다시 열심히 활동하는 걸로!👊🏻
★★코드 기여 시 다시 기여가이드를 살펴보고 추가할 사항이 있는지 체크해야겠다! 할 수 있는 건데 안하면 나만 속상하니까,,,🙁
Nextcloud/android Github: https://github.com/nextcloud/android
'android > 오픈소스 기여' 카테고리의 다른 글
contributor (0) | 2024.04.23 |
---|---|
passcode 화면에서 뒤로가기 이벤트 PR -> Merge (0) | 2024.04.23 |
passcode 레이아웃 정렬 수정 PR -> Merge (0) | 2024.04.23 |
passcode 화면에서 뒤로가기 누르면 앱 닫도록 코드 구현 후 PR (0) | 2024.04.23 |
passcode 레이아웃 정렬 변경 후 PR (0) | 2024.04.23 |