chore: add .prettierignore so editor formatting skips generated and vendored files (Closes #48) - #160
Merged
harsharajkumar-273 merged 1 commit intoJul 29, 2026
Conversation
…endored files (Closes harsharajkumar-273#48)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout. (1)
🔇 Additional comments (1)
📝 WalkthroughWalkthroughAdds a root ChangesPrettier ignore configuration
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related PRs
Suggested labels: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
harsharajkumar-273
approved these changes
Jul 29, 2026
harsharajkumar-273
merged commit Jul 29, 2026
44cd432
into
harsharajkumar-273:main
1 of 2 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #48
Problem
There is no
.prettierignoreat the repository root.Prettier is not a build dependency in this repo — it is not in
package.json,frontend/package.jsonorbackend/package.json, and there is noformatscript. It reaches contributors through theesbenp.prettier-vscodeextension that.devcontainer/devcontainer.jsoninstalls (line 23).That means anyone developing in the devcontainer has Prettier active as an editor formatter with nothing telling it what to leave alone. "Format Document" or format-on-save will happily rewrite
package-lock.json,frontend/dist/, the vendoredtest-repo/PreTeXt fixture, and the trackedbackend/dev.db, producing large unrelated diffs that reviewers then have to separate from the actual change.Change
Adds
.prettierignore, mirroring the paths already excluded by.gitignoreand.dockerignore, plus three categories those two do not need to cover:backend/dev.dbtest-repo/fixture, which is upstream contentScope
This is editor-level only. Prettier is not wired into CI or any npm script, so this PR does not change any build, lint or test behaviour — it only stops the editor integration from reformatting files that are not ours to reformat. If Prettier is later adopted as a real dependency with a
formatscript, this file is already the correct ignore set for it.Verification
No runtime code touched; no build, test or lint behaviour changes.
Summary by CodeRabbit