teamhide / fastapi-boilerplate

FastAPI boilerplate for real world production
987 stars 156 forks source link

Update `.gitignore`, remove `uvloop` in `Pipfile.lock` #7

Closed mirusu400 closed 1 year ago

mirusu400 commented 1 year ago
  1. .gitignore 가 필요한 부분만 추가해 놓으셨어서 Github에서 제공하는 기본 파이썬 .gitignore 템플릿으로 변경했습니다.

  2. 궃이 Pipfile.lockuvloop부분을 삭제했습니다. 삭제한 이유는, 현재 Pipfile을 이용해 설치 시 uvloopzipp를 설치하지 않을 뿐더러, 실제 소스코드에서도 사용하지 않고, 결정적으로 로컬 환경에서 테스트 할 때 uvloop윈도우 환경을 지원하지 않기 때문에 일부러 지웠습니다. (사실 Pipfile.lock 파일 자체를 지워도 상관없을듯 합니다)

  3. requirements.txt도 추가할려고 했는데 pipenv를 사용하시는거 같아서 다시 지웠습니다.. 개인적으로 venv나 다른 환경을 위해서 추가해주셨으면 하지만 제가 프로젝트 방향을 괜히 꺾어버리는 듯 해서 지운채로 PR 보내드립니다. (커밋은 남아 있습니다)

teamhide commented 1 year ago

.gitignore를 수정한 부분만 커밋으로 남겨서 다시 PR올려주시면 머지하겠습니다. 좋은 PR 감사합니다.