jmcheon / matcha

work in progress
0 stars 1 forks source link

fix: apply ruff and fixed code except account #54

Closed sungyongcho closed 1 month ago

sungyongcho commented 1 month ago

잠깐 적용하였었던 black formatter를 제거하고 새로운 formatter 'ruff'를 적용 하였습니다

ruff의 기본적인 설정은 back-fastapi/pyproject.toml 에서 관리되고 있습니다. 설정은 .vscode/settings.json에 적용 해 놓아서

설정:

코드 위에서 warning (노란색 밑줄) 이 안보이도록 ruff에서 포맷팅 해주는 방식으로 무조건 저장 후에 push 해서 관리하기 편하도록 쓰면 될것 같습니다

image image

이 두 부분은 따로 수정 안해놓았는데 정무님이 코드 로직을 한번 살펴보고 고친 내용으로 이 pr 위에 푸시된 코드를 확인하고 merge 진행할게요

확인 필요

image

화면에 보면 쿼리문 자체가 길어서

image

저는 이렇게 수정 해 놓았는데, 불가피 하면

image

이런식으로 #noqa: ~~~ (자동완성에 있음; disable for this line) 주석을 달아주면 되긴 하는데 최대한 안 쓰는 방향으로 가주세요

스크린샷에는 안 나와있는데 주석 달아놓고 저장 하면 warning 표시는 없어집니다

close #53