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

bugfix: lint-staged command 오류를 수정합니다. #73

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

jungwoo3490
Copy link
Member

🔥 Related Issues

✅ 작업 내용

lint-staged command 오류를 수정했어요.

Biome 공식문서를 다시 정독했는데, check command에 --format이라는 option은 존재하지 않았어요. 심지어 자동적용 옵션도 --apply가 아니라 --write였어요...

저희 논의 후에 자동적용은 끄기로 결정했으므로, --write 옵션은 추가하지 않았어요.
대신 --no-errors-on-unmatched 옵션을 추가해주었는데, glob 패턴으로 *.{js,jsx,ts,tsx} 파일들을 검사할 때, 해당 패턴과 일치하지 않는 파일이 있더라도 에러로 처리하지 않고 건너뛰도록 설정하기 위해 추가해줬어요.

추가적으로 제가 이전에 작성한 모바일 뷰 코드들에서 lint 오류가 나는 부분을 수정했어요.

@jungwoo3490 jungwoo3490 added the 🐞 bugfix 버그를 수정한 경우 label Nov 28, 2024
@jungwoo3490 jungwoo3490 self-assigned this Nov 28, 2024
Copy link

changeset-bot bot commented Nov 28, 2024

⚠️ No Changeset found

Latest commit: dcba679

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@Arooming
Copy link
Member

헙 아예 커맨드가 잘못 설정되어 있었군요 .....!! 도리가 쏘아올린 작은 공 ....
수고했서용 👍👍

@jungwoo3490
Copy link
Member Author

🚀

@jungwoo3490 jungwoo3490 merged commit 28f160f into main Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bugfix 버그를 수정한 경우 size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bugfix: lint-staged command 오류를 수정합니다.
2 participants