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.

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

Style: stylelint 삭제 #8

Merged
merged 1 commit into from
Feb 19, 2023
Merged

Style: stylelint 삭제 #8

merged 1 commit into from
Feb 19, 2023

Conversation

se030
Copy link
Contributor

@se030 se030 commented Feb 16, 2023

🤠 개요

  • Closes Style: stylelint 설정 수정 #7

  • 기존 설정이 scss 기준으로 되어있었는데, 찾아보고 이것저것 테스트해보니 emotion과 함께 사용하기 적합하지 않아 제거합니다.

💫 설명

  • @stylelint/postcss-css-in-js 패키지가 최근 deprecated 되어 emotion 환경의 stylelint 레퍼런스가 많이 없습니다.

  • 현재 emotion에서 사용할 수 있는 custom syntax 패키지가 있지만 속성을 정렬하는 플러그인은 제대로 동작하지 않고, 이미 pre-commit 훅에서 prettier와 eslint를 사용하고 있기 때문에 꼭 필요한 린팅이 아니라고 생각해 제거합니다.

    • 대신 통일성이 너무 깨지지 않게 컨벤션을 지켜서 작성해봐요!
  • 추후에 설정하고 싶다면 참고할 수 있는 레퍼런스들을 함께 첨부합니다.

📖 참고 자료

emotion과 함께 사용하기에 적합하지 않아 제거
@se030 se030 added the 💇‍♂️ style 코드 스탈 label Feb 16, 2023
@se030 se030 self-assigned this Feb 16, 2023
Copy link
Contributor

@prayinforrain prayinforrain left a comment

Choose a reason for hiding this comment

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

너무 기대했는데 아쉽네요...
style 순서에 대한 컨벤션을 찾아봐야겠어요

@iyu88 iyu88 self-requested a review February 19, 2023 12:00
Copy link
Member

@iyu88 iyu88 left a comment

Choose a reason for hiding this comment

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

LGTM
저는 걸어주신 syntax 링크 중에서 첫번째 링크가 좋아보여요
최근까지 업데이트가 있는 것 같아서요~

@iyu88 iyu88 merged commit 71827a3 into main Feb 19, 2023
@iyu88 iyu88 deleted the style/stylelint branch February 19, 2023 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💇‍♂️ style 코드 스탈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Style: stylelint 설정 수정
3 participants