Skip to content

린팅 혹은 오토포매팅 적용 #63

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.

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

Open
dongwook-chan opened this issue Aug 25, 2023 · 2 comments
Open

린팅 혹은 오토포매팅 적용 #63

dongwook-chan opened this issue Aug 25, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@dongwook-chan
Copy link

아래 사안에 대해 논의해보면 좋을 것 같습니다.

린팅 vs 오토포매팅

린터를 사용해서 PEP8 미준수 사항이 있는지 보여줄지
오토포매팅을 사용해서 PEP8을 준수하도록 강제할지

적용 시점

린터/오토포매터 적용할 시점을 커밋/PR할지

린터 및 오토포매터 종류 선택하기

어떤 린터, 오토포매터를 사용할지

@dongwook-chan dongwook-chan added the enhancement New feature or request label Aug 25, 2023
@mirageoasis
Copy link

mirageoasis commented Aug 26, 2023

https://velog.io/@qlgks1/Python-flake8-Black-%EB%8F%84%EC%9E%85-clean-code-%EC%8B%A4%EC%B2%9C%ED%95%98%EA%B8%B0

https://www.itworld.co.kr/news/162580

참고하기 좋은 글인거 같습니다. 하나 우려? 되는 점은 포메팅을 위해서 다른 의존성이 딸려오는거 같은데 이 부분은 어떻게 얘기를 하면 좋을지 여쭤봅니다.

@soulee-dev
Copy link

enhancement/lint 브랜치에서 실험중입니다.
현재 구현 사항

  • pre-commit을 이용한 ruff, black 도입
  • github actions를 통한 준수 검사

필요한 내용

  • 기여자를 위한 문서 자성
  • 미 준수 사항 수동 수정

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants