-
Notifications
You must be signed in to change notification settings - Fork 501
Update README.md #3
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
base: main
Are you sure you want to change the base?
Conversation
Added a newsletter Hello, World! in the Section: Best System design case studies. Added two article about pinterest and Uber in Section: Best System Design Tutorials Also Added newsletter Hello, World! in the Section: Best Engineering Blogs
WalkthroughThe README.md file was updated to include new curated links in the system design resources sections. Additions were made to the case studies, tutorials, and engineering blogs lists, referencing resources from "Hello, World!" and specific articles about Pinterest and Uber Eats. No existing content or structure was changed. Changes
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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
🧹 Nitpick comments (3)
README.md (3)
125-127: Validate categorization of “Hello, World!” under Case Studies“Hello, World!” is a newsletter rather than a single, in-depth case-study write-up like the other items under this heading. Consider moving it to a more appropriate section (e.g., “Best Engineering Blogs”) or explicitly linking to a specific case-study article within the newsletter to match reader expectations.
130-136: Minor style consistency for new tutorial bullets
- The two added titles are sentence-case, whereas neighbouring entries use title-case.
- A trailing question mark appears in the Pinterest title but not in the Uber Eats one; keep punctuation consistent across bullets.
Example edit:
- - [How Pinterest uses LLMs to make your Search Results more Relevant?](https://hw.glich.co/p/how-pinterest-uses-llms-to-make-your-search-results-more-relevant) + - [How Pinterest Uses LLMs to Make Your Search Results More Relevant](https://hw.glich.co/p/how-pinterest-uses-llms-to-make-your-search-results-more-relevant)(Apply similar casing to the Uber entry).
190-196: Keep engineering-blog list alphabeticalThe list is alphabetically ordered; inserting “Hello, World!” at the top breaks that ordering. Move it to the correct “H” slot:
- - [Hello, World!](https://hw.glich.co/) - [Airbnb Engineering](http://nerds.airbnb.com/) + - [Airbnb Engineering](http://nerds.airbnb.com/) + - [Hello, World!](https://hw.glich.co/)This preserves predictable ordering for readers.
|
This is actually a good resource. |
Added a newsletter Hello, World! in the Section: Best System design case studies.
Added two article about pinterest and Uber in Section: Best System Design Tutorials
Also Added newsletter Hello, World! in the Section: Best Engineering Blogs
Summary by CodeRabbit