fix: remove duplicate docstring from release mixin property

[dharshan-kumarj]

Description

This PR fixes a duplicate docstring issue in the release mixin file.

Changes

• Removed duplicate/incorrect docstring from idx_author property in backend/apps/github/models/mixins/release.py
• The property had two docstrings: one correct ("Return author for indexing.") and one incorrect ("Get top contributors.")

Related Issue

Closes #2648

[coderabbitai]

Summary by CodeRabbit

• Chores
  • Cleaned up internal documentation by removing redundant text.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Removed a redundant docstring line from the idx_author method in the release model mixin. No functional changes; documentation string cleanup only.

Changes

Cohort / File(s)	Summary
Documentation cleanup backend/apps/github/models/mixins/release.py	Removed redundant docstring line from idx_author method

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	The PR partially addresses issue #2648 by removing a duplicate docstring, but the linked issue requires adding docstrings to all property methods in multiple mixin files, which goes beyond this PR's scope.	Clarify whether this PR is a targeted fix for duplicate docstrings or if it's part of the larger effort to add docstrings to all mixin properties as required by issue #2648.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: removing a duplicate docstring from the release mixin property.
Description check	✅ Passed	The description clearly explains the duplicate docstring issue and identifies the specific file and property affected, directly relating to the changeset.
Out of Scope Changes check	✅ Passed	The changes are narrowly focused on removing a duplicate docstring from one property, which aligns with the PR's stated objective and the issue's scope.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7a08db7 and 5de8927.

📒 Files selected for processing (1)

• backend/apps/github/models/mixins/release.py (0 hunks)

💤 Files with no reviewable changes (1)

• backend/apps/github/models/mixins/release.py

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: CodeQL analysis (python)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull request overview

This PR fixes a documentation issue by removing an incorrect duplicate docstring from the idx_author property in the release model mixin.

• Removed the second, incorrect docstring ("Get top contributors.") from the idx_author property
• Retained the correct docstring ("Return author for indexing.") which accurately describes the property's purpose

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[rudransh-shrivastava]

Hey, please update the docstrings to include other property information as stated in the issue:

Mixin classes have property methods lacking docstrings. Add docstrings explaining what each property returns and its purpose.

[dharshan-kumarj]

Hey, please update the docstrings to include other property information as stated in the issue:

Mixin classes have property methods lacking docstrings. Add docstrings explaining what each property returns and its purpose.

Sure will update and let u know!

[dharshan-kumarj]

Hey, please update the docstrings to include other property information as stated in the issue:

Mixin classes have property methods lacking docstrings. Add docstrings explaining what each property returns and its purpose.

Sure will update and let u know!

I have updated this PR in the PR #2998 Mentioned here., kindly check @rudransh-shrivastava and let me know the changes or the update needed ..,please!!!!

[rudransh-shrivastava]

@dharshan-kumarj Hey, sorry for the late response. I saw the other PR (#2998). While the changes look good to me, I think it would be better if you update this one instead.
Thank you.

[dharshan-kumarj]

@dharshan-kumarj Hey, sorry for the late response. I saw the other PR (#2998). While the changes look good to me, I think it would be better if you update this one instead.
Thank you

Understood. I’ve already applied the required changes in the new PR (#2998).
If that works for you, we can close this PR.
If not, please let me know and I’ll update this new one PR instead. Can you please explain me in better way please

[rudransh-shrivastava]

@dharshan-kumarj It's better for historical context and documentation purposes. A new PR works for now, I'll take a look at #2998.

[dharshan-kumarj]

@dharshan-kumarj It's better for historical context and documentation purposes. A new PR works for now, I'll take a look at #2998.

Okiee finee, Then let me know about the merge or any updates if required in the new PR

[arkid15r]

@rudransh-shrivastava I believe this one can be merged or closed?

[rudransh-shrivastava]

Yes this one can be closed. Ill take a look at the other one now.