Skip to content

docs: add contributors section to README#154

Merged
Devnil434 merged 1 commit into
Devnil434:mainfrom
Yogender-verma:docs
Jun 17, 2026
Merged

docs: add contributors section to README#154
Devnil434 merged 1 commit into
Devnil434:mainfrom
Yogender-verma:docs

Conversation

@Yogender-verma

@Yogender-verma Yogender-verma commented Jun 16, 2026

Copy link
Copy Markdown

Changes

  • Added a Contributors section to README.md
  • Included contrib.rocks contributors badge
  • Linked badge to GitHub contributors graph

This helps recognize and showcase project contributors.

closes #151

Summary by CodeRabbit

  • Documentation
    • Added a new Contributors section in the README that acknowledges project contributors with a dedicated badge linking to the contributors graph.

@coderabbitai

coderabbitai Bot commented Jun 16, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

A new "Contributors" section is appended to README.md with a short thank-you message and a contrib.rocks badge image that links to the repository's contributors graph.

Changes

README Contributors Section

Layer / File(s) Summary
Contributors section and badge
README.md
Adds a Contributors heading, a gratitude message, and a contrib.rocks badge image linking to the repo's contributors page.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A badge for the builders, a nod for the crew,
🐇 The rabbit hops forth with a warm "Thank you!"
Their avatars gleam on the contrib.rocks wall,
A colorful quilt made by one and by all.
Now README shines bright with a welcoming glow—
Come hop in and help, let your contributions show! 🌟

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'docs: add contributors section to README' accurately and concisely summarizes the main change in the pull request.
Linked Issues check ✅ Passed The pull request successfully implements all coding requirements from issue #151: adds a Contributors section, integrates the contrib.rocks badge, and preserves existing README content.
Out of Scope Changes check ✅ Passed All changes in the pull request are directly aligned with issue #151 requirements; the Contributors section addition is the sole focus with no unrelated modifications.
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

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.

Warning

⚠️ This pull request might be slop. It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 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.

Inline comments:
In `@README.md`:
- Around line 353-358: The Contributors badge in the README uses a hard-coded
repository reference (Devnil434/Eagle) in the contrib.rocks image URL, which is
inconsistent with the placeholder pattern established elsewhere in the document
(referenced at lines 211 and 385). Replace the hard-coded repository reference
in the contrib.rocks image URL with the same placeholder pattern used throughout
the README to ensure the badge correctly points to users' own repositories when
they fork or clone the project. Check lines 211 and 385 to identify the exact
placeholder pattern being used and apply it to the repository parameter in the
contrib.rocks URL.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 147c129f-c6fc-433e-8367-f6bdd977f781

📥 Commits

Reviewing files that changed from the base of the PR and between 088dd9e and a7bfcda.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
Comment on lines +353 to +358
### 👥 Contributors

Thanks to all contributors ❤️

[![Contributors](https://contrib.rocks/image?repo=Devnil434/Eagle)](https://github.com/Devnil434/Eagle/graphs/contributors)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Hard-coded repository reference breaks README template consistency.

The contrib.rocks badge uses a hard-coded repository reference (Devnil434/Eagle) while other parts of the README use the placeholder pattern (your-org). Lines 211 and 385 show the established pattern for template customization. This inconsistency will confuse users who fork or clone the repo and expect the Contributors badge to point to their own repository.

🔧 Proposed fix to use placeholder pattern
 ### 👥 Contributors
 
 Thanks to all contributors ❤️
 
-[![Contributors](https://contrib.rocks/image?repo=Devnil434/Eagle)](https://github.com/Devnil434/Eagle/graphs/contributors)
+[![Contributors](https://contrib.rocks/image?repo=your-org/Eagle)](https://github.com/your-org/Eagle/graphs/contributors)
📝 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.

Suggested change
### 👥 Contributors
Thanks to all contributors ❤️
[![Contributors](https://contrib.rocks/image?repo=Devnil434/Eagle)](https://github.com/Devnil434/Eagle/graphs/contributors)
### 👥 Contributors
Thanks to all contributors ❤️
[![Contributors](https://contrib.rocks/image?repo=your-org/Eagle)](https://github.com/your-org/Eagle/graphs/contributors)
🤖 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 `@README.md` around lines 353 - 358, The Contributors badge in the README uses
a hard-coded repository reference (Devnil434/Eagle) in the contrib.rocks image
URL, which is inconsistent with the placeholder pattern established elsewhere in
the document (referenced at lines 211 and 385). Replace the hard-coded
repository reference in the contrib.rocks image URL with the same placeholder
pattern used throughout the README to ensure the badge correctly points to
users' own repositories when they fork or clone the project. Check lines 211 and
385 to identify the exact placeholder pattern being used and apply it to the
repository parameter in the contrib.rocks URL.

@Devnil434 Devnil434 merged commit 2ce0ae1 into Devnil434:main Jun 17, 2026
1 check passed
@Yogender-verma

Copy link
Copy Markdown
Author

@Devnil434 labels??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: add Contributors section with contrib.rocks badge

2 participants