Skip to content

Add .prettierignore to exclude build output#52

Merged
em3s merged 1 commit intokakao:mainfrom
csbiy:add-prettier-ignore
Jan 18, 2026
Merged

Add .prettierignore to exclude build output#52
em3s merged 1 commit intokakao:mainfrom
csbiy:add-prettier-ignore

Conversation

@csbiy
Copy link
Copy Markdown
Contributor

@csbiy csbiy commented Jan 17, 2026

Summary

This PR addresses the concern raised in #47 about dist files being checked by Prettier after build.

Changes

  • Added .prettierignore file to exclude build output directories
  • Excludes: dist/, .astro/, node_modules/, .cache/

Testing

  • Verified with npm run format:check - all checks pass
  • Build output directories are now properly excluded from Prettier checks

Related

- Exclude dist/ and .astro/ build directories
- Exclude node_modules/ and .cache/
- Prevents Prettier from checking generated files
@csbiy csbiy requested a review from em3s as a code owner January 17, 2026 09:49
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. module:website labels Jan 17, 2026
@em3s
Copy link
Copy Markdown
Contributor

em3s commented Jan 18, 2026

LGTM 👍

Thanks for the update. @csbiy

I’ve confirmed that the dist/ directory generated after npm run build is now properly excluded via .prettierignore.
This resolves the concern raised in #47 about Prettier running on build outputs.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 18, 2026
@em3s em3s merged commit e759b66 into kakao:main Jan 18, 2026
2 checks passed
@em3s em3s added the first-time contributor First-time contributor to this repository label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

first-time contributor First-time contributor to this repository lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants