계획했던 일 PEP8설정을 pycharm에서 적용(max length), 해당 설정을 ci에 반영, README.md에 명확한 기록 남기기 오늘 한 일 setup.cfg파일의 작성방법에 대한 글, setup.py의 작성방법에 대한 글, setuptools 공식문서를 읽었다. 내일 할 일 PEP8설정을 pycharm에서 적용(max length), 해당 설정을 ci에 반영, README.md에 명확한 기록 남기기
계획했던 일 코드리뷰를 참고해서 코드 수정하기 오늘 한 일 코드리뷰의 일부만 수정, PR를 신청했고 다시 피드백을 받았다. 개선해야할 것이 많아서 오히려 기쁘다. dataclass에 대해 공부하고 약간의 연습을 해보았다. travis-ci와 flake8 연동에 대해 고민했다. 그뿐이 아니고 travis-ci는 예전에 뭉텅이로 가져온(이미 구현된 내용물들에 이것저것 덧씌워서 차츰 만들어진 상태가 되었다) 초기 상태가 아니라 작은 것부터 차례대로 단계적인 구현이 필요하다고 느꼈다. 한꺼번에 많은 것을 건드려서 발생하게 된 에러에 많은 시간을 소비하기 보다는, 작은 것부터 하나씩 건드려서 발생하는 에러를 해결해 가는 편이 낫다는 생각이 들었다.
Continue reading
계획했던 일 이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기 오늘 한 일 코드리뷰에서 보완할 내용을 파악하고 관련 글들을 읽었다. 내일 할 일 코드리뷰를 참고해서 코드 수정하기. 금요일 밤 부터 주말 내내 연수가 있어서 조금 늦어질지도 모르겠다..
계획했던 일 이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기 오늘 한 일 레스토랑 저장소의 restaurant.py와 table.py 파일의 내용을 PEP8에 맞게 수정했다. 내일 할 일 이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기
계획했던 일 이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기 오늘 한 일 어제 Makefile을 만들고 travis-ci, codcov, coverall을 추가했는데 에러(쉘 스크립트 파일의 Permission denied)를 만지는데 시간을 많이 소비했다. 그래서 현재 문제점들을 해결하는게 급하기 때문에 최종적으로는 싸그리 초기화(…)했다. 레스토랑 저장소의 restaurant.py와 table.py를 제외한 파일의 내용을 PEP8에 맞게 수정했다. 내일 할 일 이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기
계획했던 일 코딩하기 오늘 한 일 코드리뷰를 받아보았다. 코드리뷰를 받기 위해 새로운 저장소에 그동안의 작업 내역을 작업 단위로 나누어 커밋했다. 새로 개설한 저장소에 Makefile을 만들고 travis-ci, codcov, coverall을 추가했다. 내일 할 일 ~~이슈에 제기된 문제점을 해결하여 Merge해서 Pull Request 요청하기
