Skip to content

Added contributor guidelines#26

Merged
deekshithgowda85 merged 1 commit into
prodfrom
dev
May 27, 2026
Merged

Added contributor guidelines#26
deekshithgowda85 merged 1 commit into
prodfrom
dev

Conversation

@deekshithgowda85
Copy link
Copy Markdown
Owner

@deekshithgowda85 deekshithgowda85 commented May 27, 2026

Pull Request

Summary

Describe what this PR changes in SecDev and why it is needed.

Related Issue

Closes:

What Changed

Screenshots or Demo

Add screenshots, screen recordings, or short notes if this changes the UI.

Verification

  • I ran pnpm lint
  • I manually verified the affected page, API route, or sandbox flow
  • I checked that no secrets or hardcoded credentials were added

Checklist

  • My branch name follows the repository convention
  • My commits use Conventional Commits
  • I kept the change scoped to one issue
  • I updated docs if the behavior or workflow changed
  • I am ready for review

Summary by CodeRabbit

  • Documentation
    • Added GSSoC labels and scoring scheme to the contributing guide, including point values, difficulty levels, and requirements for awarding points.
    • Updated README header layout with improved logo image positioning.

Review Change Stack

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sec-dev-3lx6 Building Building Preview, Comment May 27, 2026 12:02pm

@deekshithgowda85 deekshithgowda85 merged commit 90d3a11 into prod May 27, 2026
2 of 4 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4c5a8728-202e-457c-a1cf-5c41ba447c9f

📥 Commits

Reviewing files that changed from the base of the PR and between 88ab8cb and b12ef94.

⛔ Files ignored due to path filters (2)
  • public/image.png is excluded by !**/*.png
  • public/image1.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • CONTRIBUTING_GUIDE_GSSOC.md
  • README.md

📝 Walkthrough

Walkthrough

This PR adds a GSSoC-specific scoring guide to the contributing documentation and refines the README header layout. The changes include the project's point allocation scheme with difficulty tiers, quality/type bonuses, and award criteria, plus a minor HTML layout adjustment to the README's initial header block.

Changes

Project Documentation Updates

Layer / File(s) Summary
GSSoC scoring guide
CONTRIBUTING_GUIDE_GSSOC.md
New "GSSoC Labels & Scoring (Project-specific)" section documents the base gssoc:approved label points, difficulty tier values, optional quality multipliers, stackable type bonuses, the points formula, and the requirements for awarding points (merged PR, approved review, gssoc:approved label from maintainers).
README header layout
README.md
Adjusted the centered header markup to position both SecDev logo images within the same header block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • deekshithgowda85/SecDev#13: Both PRs modify CONTRIBUTING_GUIDE_GSSOC.md—this PR adds the specific GSSoC scoring rubric while the related PR introduces/reworks the GSSoC contribution guide itself.

Poem

🐰 The SecDev guide now shines so bright,
With scoring rules laid out just right,
GSSoC points flow like spring,
And logos dance in header's wing,
Contribution docs take flight! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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.

@coderabbitai coderabbitai Bot mentioned this pull request May 29, 2026
Merged
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.

1 participant