-
Notifications
You must be signed in to change notification settings - Fork 4
Contributing Rules
Shrikrishna Bhat edited this page Mar 19, 2025
·
1 revision
Use a consistent naming structure for branches to maintain clarity and organization. Follow this pattern:
Feature Branches:
-
feature/<short-description>
Example:feature/google-oauth
Bug Fixes:
-
fix/<issue-number>-<short-description>Example:fix/42-login-error
Hotfixes (Urgent Fixes to Production):
-
hotfix/<short-description>Example:hotfix/payment-api-timeout
Refactoring or Cleanup:
-
refactor/<short-description>Example:refactor/db-schema-optimization
-
Title: Use a concise and descriptive title. Example:
Fix: Resolved OAuth Token Expiry Issue -
Description: Include:
- The problem being fixed or feature being added
- The approach taken to solve it
- Any dependencies or changes in configuration
- Screenshots or logs if relevant
- At least one approver is required for every PR.
- Assign a team member who has expertise in the modified area.
- If the PR affects multiple domains, tag relevant reviewers accordingly.
- PRs that impact critical components require at least two approvals before merging.
-
bug→ For bug fixes -
feature→ For new features -
hotfix→ For urgent production fixes -
documentation→ For updates to README, guides, or comments -
refactor→ For code restructuring without changing functionality -
wip→ Work in progress, not ready for review yet -
tests-needed→ Requires additional tests before merging -
breaking-change→ Indicates major changes that might affect existing functionality
Each commit should be meaningful and follow the format:
<type>: <short summary>
<optional detailed description>
fix: Resolve infinite loop in game renderingfeature: Implement leaderboard APIdocs: Update README with API usagerefactor: Optimize database queries for better performance
- PRs must be reviewed and approved before merging.
- Squash merging is preferred to maintain a clean commit history.
- Ensure all tests pass before merging.
- PRs that introduce breaking changes require detailed migration steps in the PR description.
- PRs should not be merged without an explicit approval from the assigned reviewer.
- Bugs and feature requests should be logged in GitHub Issues.
- Provide a clear description, expected behavior, and relevant screenshots/logs.
- Use labels (
bug,feature,enhancement, etc.) to categorize issues correctly. - Assign issues to appropriate team members to ensure accountability.
- High-priority issues should be labeled as
urgentand assigned an ETA for resolution.
- Repository access should be restricted to authorized developers only.
- API keys, credentials, and sensitive data must not be committed to the repository.
- Always review third-party dependencies before adding them to the project.
- Enable branch protection rules to prevent unauthorized merges into
mainorproductionbranches.
- Introduction
- For Everyone
- For Developers
- For Designers
- For Management
- Introduction
- For Everyone
- For Developers
- For Designers
- For Management
- Banner
- Other Reusable Components (to be added)
- Introduction
- For Everyone
- For Developers
- For Designers
- For Management
- Resources for a New Developer
- Resources for a New Designer
- Resources for a New Project Manager