Skip to content
This repository was archived by the owner on Sep 21, 2026. It is now read-only.

Fix typos in blog - #2828

Merged
adityaoberai merged 2 commits into
mainfrom
blog-fixes
Mar 23, 2026
Merged

adityaoberai merged 2 commits into
mainfrom
blog-fixes

Conversation

@adityaoberai

@adityaoberai adityaoberai commented Mar 23, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • Documentation
    • Corrected formatting issues in blog posts to ensure proper code block structure and content display.

@coderabbitai

coderabbitai Bot commented Mar 23, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e765a217-b20e-4013-8b77-267d88db829c

📥 Commits

Reviewing files that changed from the base of the PR and between a3fa21f and ccc85ac.

📒 Files selected for processing (2)
  • src/routes/blog/post/appwrite-vs-cloudinary/+page.markdoc
  • src/routes/blog/post/easiest-file-uploads/+page.markdoc
💤 Files with no reviewable changes (1)
  • src/routes/blog/post/appwrite-vs-cloudinary/+page.markdoc

Walkthrough

Two minor syntax corrections were made to Markdoc blog post files. An extraneous trailing line was removed from the appwrite-vs-cloudinary post file, and a missing closing code fence was added to the easiest-file-uploads post file to properly terminate a JavaScript code block. Both changes correct formatting issues in the documentation files without altering content or logic.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'Fix typos in blog' is vague and does not accurately reflect the actual changes, which involve fixing a missing code fence and removing extraneous text, not correcting typos. Revise the title to be more specific about the actual changes, such as 'Fix markdown formatting in blog posts' or 'Fix code block fence and remove trailing text in blog'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch blog-fixes

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps

greptile-apps Bot commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes two content issues in blog post Markdoc files: a missing closing code fence in easiest-file-uploads that would have broken rendering of all subsequent sections, and a stray appwrite- artifact at the end of appwrite-vs-cloudinary that had no trailing newline.

  • easiest-file-uploads/+page.markdoc: Adds the missing closing ``` for the createBucket code block, preventing the "Uploading a file from the browser" heading and all content below it from being incorrectly rendered as code.
  • appwrite-vs-cloudinary/+page.markdoc: Removes a dangling appwrite- string (with no newline) appended after the final links list — a clear leftover artifact.

Confidence Score: 5/5

  • This PR is safe to merge — both changes are minimal, targeted fixes to blog post content.
  • Both changes are straightforward documentation/content corrections with no logic, no code execution, and no risk of regressions. The missing code fence fix is particularly impactful for rendering correctness.
  • No files require special attention.

Important Files Changed

Filename Overview
src/routes/blog/post/appwrite-vs-cloudinary/+page.markdoc Removes a dangling appwrite- artifact at the end of the file (which also lacked a trailing newline). The fix is correct and leaves the document ending cleanly after the links list.
src/routes/blog/post/easiest-file-uploads/+page.markdoc Adds the missing closing ``` fence for the createBucket code block. Without it, the "Uploading a file from the browser" section and all subsequent content would have been swallowed into the unclosed code block, breaking the blog post rendering.

Reviews (1): Last reviewed commit: "Remove unnecessary text" | Re-trigger Greptile

@adityaoberai
adityaoberai merged commit 94418f2 into main Mar 23, 2026
6 checks passed
@adityaoberai
adityaoberai deleted the blog-fixes branch March 23, 2026 12:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants