lint: run prettier on packages/sdk-core/src/lib files#118
Conversation
|
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe PR removes the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates Prettier scope so lib directories are no longer globally ignored, and applies Prettier formatting to the SDK core crypto utilities (and their tests).
Changes:
- Removed
**/libfrom.prettierignoreso Prettier runs onsrc/libcontent. - Applied Prettier formatting to
packages/sdk-core/src/lib/crypto.ts. - Applied Prettier formatting to
packages/sdk-core/tests/lib/crypto.test.ts.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
packages/sdk-core/src/lib/crypto.ts |
Prettier-only formatting of crypto helpers (stableStringify, sha256Hash). |
packages/sdk-core/tests/lib/crypto.test.ts |
Prettier-only formatting of crypto unit tests. |
.prettierignore |
Stops ignoring all lib directories so src/lib files are included in formatting checks. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Note
Updates code style and tooling while adding coverage for crypto utilities.
packages/sdk-core/src/lib/crypto.tswith Prettier (no functional changes)stableStringifybehavior andsha256Hashvalidation errors inpackages/sdk-core/tests/lib/crypto.test.ts.prettierignoreentries (logs/lockfiles/pnpm store) and remove**/libfrom ignore listWritten by Cursor Bugbot for commit ca03f57. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
Style
Chores
✏️ Tip: You can customize this high-level summary in your review settings.