-
Notifications
You must be signed in to change notification settings - Fork 1
fix: typo #25
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
fix: typo #25
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Rohit Goswami seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
WalkthroughEdits to a single MDX blog post adjust punctuation in one paragraph and change formatting of two monetary amounts to inline code styling. No logic, metadata, or exports were modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Pre-merge checks (1 passed, 1 warning, 1 inconclusive)❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
Poem
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. ✨ Finishing Touches🧪 Generate unit tests
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
content/posts/deepfake-check.mdx (1)
29-29: Add descriptive alt text to images (accessibility).Both images render without alt text. Provide succinct descriptions to improve screen-reader support and SEO.
- + - +Also applies to: 109-109
🧹 Nitpick comments (2)
content/posts/deepfake-check.mdx (2)
47-47: Use an em dash for smoother cadence (style nit).Change “private data, just a” to “private data—just a” for cleaner flow and to match common editorial style.
-Companies wouldn’t need to see the private data, just a cryptographic confirmation: *this is a real person, not an AI deepfake.* +Companies wouldn’t need to see the private data—just a cryptographic confirmation: *this is a real person, not an AI deepfake.*
63-63: Avoid code styling for currency amounts (semantics/accessibility).Inline code implies code, not money, and can confuse screen readers. Prefer plain text or bold/italics.
-Eventually, she sent over `$81,000` in cash and even sold her condo worth `$350,000`, all to fund her “relationship.” +Eventually, she sent over $81,000 in cash and even sold her condo worth $350,000, all to fund her “relationship.”
Description
Testing (ignore for documentation update)
Type of change
Checklist:
Additional Notes:
Summary by CodeRabbit