dramaling-app/.github/PULL_REQUEST_TEMPLATE.md

2.5 KiB

Pull Request

📋 Summary

🎯 Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🏗️ Infrastructure/build changes
  • 🧹 Code cleanup/refactoring
  • 🧪 Tests only

🛠 Changes Made

Frontend (Flutter)

  • UI components updated
  • State management changes
  • Navigation changes
  • New screens/widgets added

Backend (.NET Core)

  • API endpoints added/modified
  • Database schema changes
  • Business logic updates
  • Authentication/authorization changes

🧪 Testing

Flutter Testing

  • Unit tests added/updated
  • Widget tests added/updated
  • Integration tests added/updated
  • Manual testing completed on iOS
  • Manual testing completed on Android

.NET Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • API testing completed
  • Database migration tested

📱 Screenshots/Videos

📝 Additional Notes

Checklist

Code Quality

  • Code follows the established coding standards
  • Self-review of the code completed
  • Code is properly commented (especially complex logic)
  • No debugging code or console logs left in
  • Error handling is appropriate

Documentation

  • Documentation updated (if needed)
  • API documentation updated (if applicable)
  • README updated (if needed)

Security & Performance

  • No sensitive data exposed in code
  • Performance impact considered
  • Security implications reviewed
  • Accessibility guidelines followed (for UI changes)

Testing & Deployment

  • All tests pass locally
  • CI/CD pipeline passes
  • Database migrations work (if applicable)
  • Feature works in staging environment

👥 Reviewers

@team-leads @senior-developers


Note: Please ensure all checkboxes are marked before requesting review.