[add] a new blog#392
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughUpdates Twitter and Instagram profile URLs in the blog socials component and in one existing blog post, and adds a new MDX blog post titled "why 100% software isn't good" covering OSS project labeling concepts, tag-accuracy challenges, and a direction toward opensox pro. ChangesBlog Content and Social URL Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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.
🧹 Nitpick comments (2)
apps/web/src/app/(main)/blog/blog-socials.tsx (1)
3-40: ⚡ Quick winConsider renaming the
socialsarray toSOCIALSto follow UPPER_SNAKE_CASE convention for constants.Per coding guidelines, constants should use UPPER_SNAKE_CASE. Since this is a module-level constant, renaming would align with the convention.
✨ Proposed refactor
-const socials = [ +const SOCIALS = [ { label: "Twitter", ~ { label: "YouTube", ~ { label: "Instagram", ~ { label: "Discord", ~ ]; export default function BlogSocials() { return ( <div className="flex items-center gap-3"> - {socials.map((s) => ( + {SOCIALS.map((s) => (🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@apps/web/src/app/`(main)/blog/blog-socials.tsx around lines 3 - 40, Rename the `socials` constant to `SOCIALS` to adhere to the UPPER_SNAKE_CASE naming convention for module-level constants. Update the variable declaration and any references to this constant throughout the file to use the new uppercase name.Source: Coding guidelines
apps/web/src/content/blog/why-100-percent-software-isnt-good.mdx (1)
9-11: 💤 Low valueApply standard hyphenation for compound adjectives before nouns.
Per grammar conventions, compound adjectives that precede nouns should be hyphenated. Several instances need fixing:
- Line 9: "open source projects" → "open-source projects"
- Line 11: "open source contributor" → "open-source contributor" and "open source project" → "open-source project"
Consider applying the same fix to line 43 ("high-level architecture") and line 79.
🔧 Proposed fixes
-what if i scrap all the open source projects, filter them and then manually labels them like "this oss project hires contributors", "this project is active", "this project has great maintainers". +what if i scrap all the open-source projects, filter them and then manually labels them like "this oss project hires contributors", "this project is active", "this project has great maintainers". -so that a new open source contributor doesn't have to waste weeks of his/her time to find a suitable open source project to contribute. +so that a new open-source contributor doesn't have to waste weeks of his/her time to find a suitable open-source project to contribute.Also apply similar fixes to:
- Line 43: "high level architecture" → "high-level architecture"
- Line 79: "open source projects" → "open-source projects"
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@apps/web/src/content/blog/why-100-percent-software-isnt-good.mdx` around lines 9 - 11, Apply standard hyphenation rules to compound adjectives that precede nouns in the markdown file. Specifically, replace all instances of "open source" with "open-source" and "high level" with "high-level" where these phrases appear as adjectives before nouns (such as in "open-source projects", "open-source contributor", and "high-level architecture"). This applies to all occurrences throughout the file where compound adjectives precede their nouns.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@apps/web/src/app/`(main)/blog/blog-socials.tsx:
- Around line 3-40: Rename the `socials` constant to `SOCIALS` to adhere to the
UPPER_SNAKE_CASE naming convention for module-level constants. Update the
variable declaration and any references to this constant throughout the file to
use the new uppercase name.
In `@apps/web/src/content/blog/why-100-percent-software-isnt-good.mdx`:
- Around line 9-11: Apply standard hyphenation rules to compound adjectives that
precede nouns in the markdown file. Specifically, replace all instances of "open
source" with "open-source" and "high level" with "high-level" where these
phrases appear as adjectives before nouns (such as in "open-source projects",
"open-source contributor", and "high-level architecture"). This applies to all
occurrences throughout the file where compound adjectives precede their nouns.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 7e5850d7-a6b2-41a0-8aa0-3b2c0a0ad472
📒 Files selected for processing (3)
apps/web/src/app/(main)/blog/blog-socials.tsxapps/web/src/content/blog/how-i-started-opensox.mdxapps/web/src/content/blog/why-100-percent-software-isnt-good.mdx
Summary by CodeRabbit
Documentation
Chores