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

[Fix] postcss.config를 es module 문법으로 변경 #107

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

simeunseo
Copy link
Collaborator

@simeunseo simeunseo commented Nov 5, 2024

관련 이슈 번호

close #106

작업 내용

package.json의 "type": "module"설정으로 프로젝트 내 js 파일이 es module로 처리되고 있지만
postcss.config에서 common js 문법으로 작성하여 오류가 나는 상황이었습니다.
따라서 postcss.config를 es module문법으로 수정하였습니다.

@simeunseo simeunseo added 🛠️ Fix 오류 수정 작업 FE Frontend 관련 작업 labels Nov 5, 2024
@simeunseo simeunseo added this to the 통합 개발 환경 구축 milestone Nov 5, 2024
@simeunseo simeunseo self-assigned this Nov 5, 2024
Copy link
Collaborator

@seoko97 seoko97 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
혹시 .cjs 확장자 변경은 고려하지 않으신 이유가 있을까요?

@simeunseo
Copy link
Collaborator Author

simeunseo commented Nov 5, 2024

@seoko97

config 파일끼리 일관성이 있으면 좋을 것 같았는데,
마침 tailwind.config와 vite.config가 둘다 es module로 작성되어있어서
통일성을 부여하기위해 이렇게 작업했습니다!

그런데 각 설정 파일의 특성을 고려해서 어울리는 문법을 쓰는게 더 적절하다고도 생각이 드는데
postcss의 경우 cjs로 바꾸는게 좋은지, es module로 작성하면 좋을지
판단기준을 잘 모르겠네요 🧐

@simeunseo simeunseo merged commit e68c771 into develop Nov 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE Frontend 관련 작업 🛠️ Fix 오류 수정 작업 size/xs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] postcss.config ES Module 충돌
3 participants