-
-
Notifications
You must be signed in to change notification settings - Fork 323
RajGupta as a mentor #5391
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?
RajGupta as a mentor #5391
Conversation
|
👋 Hi @Rajgupta36! This pull request needs a peer review before it can be merged. Please request a review from a team member who is not:
Once a valid peer review is submitted, this check will pass automatically. Thank you! |
WalkthroughA mentor profile card for Raj Gupta was added to the GSoC 2026 mentors section in the website template. The card is duplicated in the same section, appearing twice with identical content including name, role, and description. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ 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 |
📊 Monthly LeaderboardHi @Rajgupta36! Here's how you rank for January 2026:
Leaderboard based on contributions in January 2026. Keep up the great work! 🚀 |
❌ Pre-commit checks failedThe pre-commit hooks found issues that need to be fixed. Please run the following commands locally to fix them: # Install pre-commit if you haven't already
pip install pre-commit
# Run pre-commit on all files
pre-commit run --all-files
# Or run pre-commit on staged files only
pre-commit runAfter running these commands, the pre-commit hooks will automatically fix most issues. 💡 Tip: You can set up pre-commit to run automatically on every commit by running: pre-commit installPre-commit outputFor more information, see the pre-commit documentation. |
mdkaifansari04
left a comment
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.
LGTM
|
@Rajgupta36 Please run |
S3DFX-CYBER
left a comment
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.
Hi @Rajgupta36, welcome to the mentor list! I noticed a couple of things that need fixing before this can be merged:
1.Duplicate Card: It looks like the mentor card was added twice in gsoc.html. Could you remove the extra one?
2.Formatting: The pre-commit check failed on djLint. You can fix this by running pre-commit run --all-files locally and pushing the changes.
I'll be happy to approve once these are cleared up!
Can you please describe which card is added twice in gsoc.html. |
|
hey @mdkaifansari04 I would like to clarify your doubt as The card added twice is the 'OWASP BLT & GSOC Partnership' section. It first appears at line 153, immediately following the Program Benefits. It is then repeated at line 587, right after the Mentor profiles. You can check yourself . Thanks!!! |
|
@S3DFX-CYBER But the pr does not have changes not in line 153 or 587. Can you verify this |
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.