banksalad / python

🍪 🐍 Use this template if you love Python

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

travis 옵션 추가

winterjung opened this issue · comments

  • .travis.yml with python version
  • badge도 달 수 있으나 private repo같은 경우 token 값을 추가해줘야합니다.
    • e.g. ![Build Status](https://api.travis-ci.com/Rainist/private-repo.svg?token=travis-token&branch=develop) (이미지를 감싸는 링크는 생략했습니다)
    • token 없이 기본 뱃지를 달아두고 README 업데이트 때 갱신하면 어떨까 합니다 e.g. ![Build Status](https://travis-ci.com/Rainist/private-repo.svg)
  • codecov도 travis가 yes라면 기본으로 포함시킬지? 아니면 choices로 제공할 수도 있구요. 얘도 token이 필요한데 위와 같이 생략하는 형태면 어떨까 합니다

@sunghyunzz

  1. 우선 Travis CI 템플릿에 추가하는 것 자체는 매우 좋은 것 같습니다 👍
  2. badge의 경우 https://badgen.net/travis/babel/babel 와 같이 프로젝트와 무관한 dummy badge를 달아두고 템플릿 이용자가 변경할 수 있도록 안내하는 방식이 좋을 것 같아요. .com.org 구분도 그렇고 말씀하신대로 private의 경우 token 등도 있기 때문에 완전히 nice하게 템플릿 제공이 어려울 것 같습니다.
  3. 저희 템플릿 현재 모습 상 coverage 측정이 선택이 아닌 필수이기 때문에 use_travis [y/n]을 입력 받고, 사용할 경우에는 codecov 관련 내용을 자동으로 after_success 항목에 추가한 뒤 사용자가 불필요할 경우 삭제하는 방향으로 유도해도 충분할 것 같습니다.
  4. Travis CI 사용 옵션의 기본 선택지는 y이면 좋을 것 같습니다.
  • root level Travis CI status badge image URL 바꾸는 일도 해야합니다.

그냥 클릭시 이동하는 URL의 경우 스크릿모드로 public repo인 Rainist/rrn을 확인했을 땐 현재 연결된 .com형태인 https://travis-ci.com/Rainist/rrn.org형태인 https://travis-ci.org/Rainist/rrn 모두 잘 동작하는데 이것도 바꿔줘야할까요?

이미지 URL의 경우는 badgen.net 에서 api.travis.org 형태로 변경하겠습니다.

#19 를 통해 URL 변경했으며 repo를 public으로 전환하면 적용됩니다.