GitHub CLI에 대한 나의 첫 번째 오픈 소스 기여
5812 단어 goopensource
이 기여에 대한 생각
나의 최근 오픈 소스 기여는 live streaming on twitch 동안 발생했습니다. 때때로 스트림이 끝날 때 새로운GitHub CLI을 활용하여 프로젝트 기여자를 나열하고 CLI가 봇의 기여를 필터링하지 않는 것을 확인했습니다.
내 프로젝트의 최고 유지 관리자는 dependsabot-preview[bot] 및 gh-action[bot]이며, 그들의 작업에 감사하는 만큼 "감사 크레딧"에 나열되지 않은 봇을 선호합니다.
이 문제에 대해서는 첫 번째 기여를 할 만큼 충분한 관심이 있었습니다.
내 기여도 설정
코드를 작성하기 전에 이 문제에 대한 수정 사항을 조사하기 위해 저장소를 분기했습니다. 이 필터를 추가하는 프로젝트를 알게 된 후 수정 사항으로 바로 뛰어들 뻔했지만 유지 관리자의 응답을 기다리는 방법으로 제안된 솔루션에 대한 문제를 열기로 선택했습니다.
CONTRIBUTING.md는 기여자가 기여를 하기 전에 문제를 열도록 요청합니다.
내 문제 설명에는 인간 전용으로 필터링하는 옵션과 함께
gh credits
명령을 실행하려는 욕구가 포함되어 있습니다. 아래에서 해당 문제를 읽을 수 있습니다.크레딧에서 봇을 필터링하는 기능
#1597
bdougie
에 게시됨
해결하려는 기능이나 문제를 설명하십시오.
내 프로젝트의 최고 관리자는 dependsabot-preview[bot] 및 gh-action[bot]입니다. 이 문제를 선제적으로 여는 중입니다.
기고자 회의에서 gh credits
명령을 실행하고 싶지만 인간 전용으로 필터링하는 옵션이 있습니다. TBH, 인간 전용 필터를 기본값으로 만들고 봇 표시를 옵션으로 만들고 싶습니다.
제안 된 해법
CLI와 해당 사용자에게 어떤 이점이 있습니까?
나는 dependsabot이 목록에서 1위 기여자가 되는 것이 나 또는 다른 사람들에게 유용하다고 생각하지 않으며 기본적으로 인간 전용을 강조하는 것이 현재로서는 또는 적어도 봇이 인계받을 때까지 선호됩니다.
옵션 사용 예시gh credits open-sauced/open-sauced --bots
플래그 없이 크레딧을 실행하면 사람만 반환됩니다.gh credits open-sauced/open-sauced
나는 코브라를 처음 접하고 Go의 초보자이지만 이것이 플래그입니다. cmd.Flags().BoolVarP(&opts.Bots, "bots", "bts", false, "Include bots in the credits")
의문
이것이 필요합니까?
나는 grep을 사용하여 인간 전용 이름을 얻을 수 있다는 것을 알고 있지만 그렇게 할 수 없고 멋진 애니메이션을 얻을 수 있습니다.gh credits open-sauced/open-sauced | grep -v '\[bot\] $'
추가 컨텍스트
현재 출력 결과의 스크린샷(봇 포함).
View on GitHub
열성적인 기여자로서 기여에 대한 관심을 알리는 이 단계를 놓치기 쉽습니다. 나는 이전에 프로젝트에 대한 기여라고 생각한 것을 만들었지만 프로젝트가 다른 방향으로 가고 있고 내가 커밋한 코드가 향후 삭제하도록 표시된 섹션에 있다는 것을 알게 되었습니다.
내 기여가 수락되었습니다.
내 솔루션 작업을 시작하기 위해 첫 번째 응답을 받기까지 몇 시간 동안 기다렸다가 엄지손가락을 치켜세우기까지 며칠이 더 걸렸습니다. 관리자는 친절했고 스스로 제안과 경계 솔루션을 제공했습니다. 우리는 내 대답이 가깝지만 단순화될 수 있다고 결론을 내렸습니다. 이것이 문제를 먼저 여는 것이 선호되는 첫 번째 기여인 또 다른 이유입니다.
아래에서 병합된 솔루션을 볼 수 있습니다.
인간을 위한 gh 크레딧 필터링
#1623
bdougie
에 게시됨
이게 뭔가요?
내 프로젝트의 최고 관리자는 dependsabot-preview[bot] 및 gh-action[bot]입니다. https://github.com/cli/cli/issues/1597에 대한 응답으로 이 PR을 시작합니다.
해결책
사람(사용자)이 아닌 로그인을 제외하기 위해 다른 조건을 추가했습니다.
Codespace에서 테스트를 실행했는데 통과했습니다. 이 조건에 대해 새 테스트가 필요한지 알려주세요.
스크린샷
봇 없음
View on GitHub
저의 오픈 소스 여정에 함께 해주세요
저는 금요일 Twitch 스트림을 오픈 소스 작업에 바치고 오픈 소스 기여에 대한 아이디어를 통해 대화를 나누는 사전 스트림 아마스에 여러분을 초대합니다.
에서 전체 비디오를 시청하세요.
Reference
이 문제에 관하여(GitHub CLI에 대한 나의 첫 번째 오픈 소스 기여), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다
https://dev.to/opensauced/my-first-open-source-contribution-to-the-github-cli-5g27
텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념
(Collection and Share based on the CC Protocol.)
bdougie
에 게시됨
이게 뭔가요?
내 프로젝트의 최고 관리자는 dependsabot-preview[bot] 및 gh-action[bot]입니다. https://github.com/cli/cli/issues/1597에 대한 응답으로 이 PR을 시작합니다.
해결책
사람(사용자)이 아닌 로그인을 제외하기 위해 다른 조건을 추가했습니다.
Codespace에서 테스트를 실행했는데 통과했습니다. 이 조건에 대해 새 테스트가 필요한지 알려주세요.
스크린샷
봇 없음
View on GitHub
저의 오픈 소스 여정에 함께 해주세요
저는 금요일 Twitch 스트림을 오픈 소스 작업에 바치고 오픈 소스 기여에 대한 아이디어를 통해 대화를 나누는 사전 스트림 아마스에 여러분을 초대합니다.
에서 전체 비디오를 시청하세요.
Reference
이 문제에 관하여(GitHub CLI에 대한 나의 첫 번째 오픈 소스 기여), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다
https://dev.to/opensauced/my-first-open-source-contribution-to-the-github-cli-5g27
텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념
(Collection and Share based on the CC Protocol.)
사람(사용자)이 아닌 로그인을 제외하기 위해 다른 조건을 추가했습니다.
Codespace에서 테스트를 실행했는데 통과했습니다. 이 조건에 대해 새 테스트가 필요한지 알려주세요.
스크린샷
봇 없음
View on GitHub
저의 오픈 소스 여정에 함께 해주세요
저는 금요일 Twitch 스트림을 오픈 소스 작업에 바치고 오픈 소스 기여에 대한 아이디어를 통해 대화를 나누는 사전 스트림 아마스에 여러분을 초대합니다.
에서 전체 비디오를 시청하세요.
Reference
이 문제에 관하여(GitHub CLI에 대한 나의 첫 번째 오픈 소스 기여), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다
https://dev.to/opensauced/my-first-open-source-contribution-to-the-github-cli-5g27
텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념
(Collection and Share based on the CC Protocol.)
Reference
이 문제에 관하여(GitHub CLI에 대한 나의 첫 번째 오픈 소스 기여), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다 https://dev.to/opensauced/my-first-open-source-contribution-to-the-github-cli-5g27텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념 (Collection and Share based on the CC Protocol.)