docs: add Rahul Kumar to contributors#521
docs: add Rahul Kumar to contributors#521Rahulkumar65 wants to merge 2 commits intoappwrite:masterfrom
Conversation
WalkthroughA single bullet point is appended to Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
Greptile SummaryThis PR appends a personal contributor bio line to Key issues:
Confidence Score: 1/5This PR is not ready to merge — the entry is placed in the wrong section and does not meet the file's formatting conventions. The change adds content to an incorrect section of the README and does not follow the link-based formatting used throughout the rest of the file. The PR description is also entirely empty. The change needs to be restructured or removed before merging. README.md — wrong section placement and missing link/newline. Important Files Changed
Reviews (1): Last reviewed commit: "docs: add Rahul Kumar to contributors" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
README.md
Outdated
| You can also find the [Appwrite Topic on GitHub](https://github.com/topics/appwrite)! | ||
|
|
||
| - [DEV Community](https://dev.to/appwrite) | ||
| - Rahul Kumar 🚀 — Learning Open Source Contribution No newline at end of file |
There was a problem hiding this comment.
This line is appended inside the Communities section (which lists community links like Discord, Twitter, DEV Community, etc.), not a contributors section. A plain-text personal bio entry does not belong here.
If the intent is to add yourself as a contributor to the project, this repository uses GitHub's native contributor graph (driven by commits/PRs). A self-promotional line in the Communities list is not the correct way to do so.
If the project does maintain a curated contributors section, the entry should:
- Be placed under the correct heading (which does not currently exist in this file).
- Include a link to the contributor's GitHub profile or relevant work, consistent with every other entry in the file.
README.md
Outdated
| You can also find the [Appwrite Topic on GitHub](https://github.com/topics/appwrite)! | ||
|
|
||
| - [DEV Community](https://dev.to/appwrite) | ||
| - Rahul Kumar 🚀 — Learning Open Source Contribution No newline at end of file |
There was a problem hiding this comment.
The added line has no trailing newline (\ No newline at end of file in the diff). This breaks the POSIX file convention and can cause issues with tooling that expects a newline-terminated file.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 556: Replace the plain text list item "Rahul Kumar 🚀 — Learning Open
Source Contribution" with a markdown list entry matching the existing
Communities pattern: make it a hyphen bullet, hyphenate "Open-Source", and
convert the name/phrase into a markdown link (e.g., "- [Rahul Kumar 🚀 —
Learning-Source Contribution](<URL>)") so it visually and structurally matches
other entries; ensure the link target is set to the appropriate URL for the
resource.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
README.md
Outdated
| You can also find the [Appwrite Topic on GitHub](https://github.com/topics/appwrite)! | ||
|
|
||
| - [DEV Community](https://dev.to/appwrite) | ||
| - Rahul Kumar 🚀 — Learning Open Source Contribution No newline at end of file |
There was a problem hiding this comment.
Make this new list item consistent with the Communities format.
Line 556 is plain text while the rest of this section uses markdown links, and the phrase should be hyphenated as “Open-Source” for consistency.
Suggested edit
-- Rahul Kumar 🚀 — Learning Open Source Contribution
+- [Rahul Kumar 🚀 — Learning Open-Source Contribution](https://github.com/Rahulkumar65)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - Rahul Kumar 🚀 — Learning Open Source Contribution | |
| - [Rahul Kumar 🚀 — Learning Open-Source Contribution](https://github.com/Rahulkumar65) |
🧰 Tools
🪛 LanguageTool
[uncategorized] ~556-~556: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...o/appwrite) - Rahul Kumar 🚀 — Learning Open Source Contribution
(EN_COMPOUND_ADJECTIVE_INTERNAL)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 556, Replace the plain text list item "Rahul Kumar 🚀 —
Learning Open Source Contribution" with a markdown list entry matching the
existing Communities pattern: make it a hyphen bullet, hyphenate "Open-Source",
and convert the name/phrase into a markdown link (e.g., "- [Rahul Kumar 🚀 —
Learning-Source Contribution](<URL>)") so it visually and structurally matches
other entries; ensure the link target is set to the appropriate URL for the
resource.
1 participant
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