Skip to content
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

black formatting #39

Merged
merged 6 commits into from
Apr 6, 2024
Merged

black formatting #39

merged 6 commits into from
Apr 6, 2024

Conversation

mqjinwon
Copy link
Collaborator

@mqjinwon mqjinwon commented Apr 2, 2024

Black formatting

변경 사항 요약

  • 쉽게 사용할 수 있는 포매팅 방식을 차용
  • 본 포매터를 통해, 일관성 있는 코딩을 하고 파일의 변경을 최소화 시킴

테스트 방법

  • black *

@mqjinwon mqjinwon self-assigned this Apr 2, 2024
harderthan
harderthan approved these changes Apr 3, 2024
View reviewed changes
Copy link
Collaborator

@harderthan harderthan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문인데요.

black은 포매팅 설정 파일을 이렇게 관리하나요? 쓰려고 하시는 설정 파일을 하나 올려주실 수 있을까요?

이것과 무관하게 PR은 Approve 눌러둘게요.

@mqjinwon
Copy link
Collaborator Author

mqjinwon commented Apr 6, 2024

아 default black configuration를 사용하려고 했었습니다.
default값이 사용자가 다른지와 설정파일을 꼭 사용해야하는 지는 한번 알아보겠습니다.

@mqjinwon mqjinwon merged commit f3b54eb into AuTURBO:main Apr 6, 2024
1 check passed
@mqjinwon mqjinwon deleted the format/black branch April 6, 2024 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harderthan harderthan harderthan approved these changes

Assignees

@mqjinwon mqjinwon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mqjinwon @harderthan