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

chore(*): pre-push .next 파일 제거하도록 추가 #197

Merged
merged 2 commits into from
Sep 10, 2024
Merged

Conversation

Doeunnkimm
Copy link
Member

@Doeunnkimm Doeunnkimm commented Sep 7, 2024

🎉 변경 사항

pre-push 시 .next 파일을 제거하도록 추가합니다.

🔗 링크

🙏 여기는 꼭 봐주세요!

Comment on lines +4 to +6
echo "🗑️ .next 디렉터리를 삭제합니다."
find . -type d -name ".next" -exec rm -rf {} +

Copy link
Collaborator

Choose a reason for hiding this comment

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

엇 혹시 이걸 pre-push에 추가하신 이유가 어떻게 되시나요??

기존 잔여 .next로 인한 빌드 오류로 push가 안 되는 현상 때문이실까요? 지금은 이 방법으로 해도 좋을 것 같은데, 개인적인 생각으론 next 설정 자체에서 캐시 관련 옵션을 제거해서 local dev 띄울 때나 build할 때 캐시된 항목을 제거하는 방향이 좋다고 생각합니다..

만 빠른 시일 내에 제가 할 자신은 없어서 우선 어프로브 가겠습니다. 😀

Copy link
Member Author

Choose a reason for hiding this comment

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

오~! 빌드할 때 캐시 지우는게 제일 좋아보이긴 하네요 👍🏽

추가한 이유는 말씀하신 이유 맞아용 push할 때 캐시 파일 때문에 에러 나는 경험 때문 맞습니다

음~ 듣고나니 고민되네요

@Doeunnkimm Doeunnkimm merged commit bb4654e into main Sep 10, 2024
2 checks passed
@Doeunnkimm Doeunnkimm deleted the husky-next branch September 10, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants