search

lewisleedev / thecampy

더캠프 인터넷편지 파이썬 라이브러리
https://pypi.org/project/thecampy/
MIT License
22 stars 4 forks source link

부대이름 항목 삭제 #6

Closed SyphonArch closed 3 years ago

SyphonArch commented 3 years ago

현재 README 파일에서는, 라이브러리의 사용을 위해 정확한 부대 이름이 필요하다는 내용이 있습니다.

하지만 이는 더 이상 사실이 아닌 듯 합니다. (the-camp-lib의 약 1년 전 업데이트 참고: https://github.com/parksb/the-camp-lib/commit/6541f843ac45e62cd7038b7b673d841d78da6507)

부대 이름을 입력하지 않아도 코드가 동작할 수 있도록, models.Soldier의 생성자에서 unit값을 optional하게 변경하였습니다. 또한 README.md에서 부대명에 대한 언급을 삭제했습니다. unit값이 기존과 같이 입력될 상황을 대비해, 기존의 코드는 삭제하지 않아 이전 코드와의 호환성을 남겨두었습니다.

본 변경사항 없이도 라이브러리가 정상동작하긴 하나, 라이브러리 사용자들이 부대이름 값을 어떻게 설정해야 하는지 불필요하게 고민하는 부분을 줄일 수 있을 것 같아 PR 올립니다.

lewisleedev commented 3 years ago

수정 감사합니다. merge후 Release하겠습니다.

추가 수정사항이나 개선사항 있다면 PR부탁드립니다.