New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

코드 리뷰 #1

Open

juwoong wants to merge 2 commits into main from review

Contributor

juwoong commented Jan 7, 2021

.gitignore 파일을 추가했습니다
- 해당 파일 내에 명시된 파일/폴더를 git에서 무시합니다.
- 생성을 위해 https://gitignore.io/ 를 이용하면 편합니다.
변수의 컨벤션을 Snake case로 통일했습니다.
- 앞으로 코드를 작성할 때, 네이밍 컨벤션을 하나로 통일해주세요. 파이선에서는 스네이크 케이스를 권장하고 있습니다.
해당 프로그램의 dependency를 관리해 주세요.
- requirements.txt / pipenv / poetry 등의 선택지가 있습니다
- venv는 찾아보시고 꼭 적용해 주세요

juwoong added 2 commits

January 7, 2021 23:42


          🙈 add .gitignore

ff67626


          🎨 add format

189af51

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet