Skip to content

Enhancement: Add PR template with security and performance checklists#711

Merged
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
AugistineCreates:feat/pr-template
Jun 27, 2026
Merged

Enhancement: Add PR template with security and performance checklists#711
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
AugistineCreates:feat/pr-template

Conversation

@AugistineCreates

@AugistineCreates AugistineCreates commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

This pr closes #671

Summary

This PR introduces a standardized Pull Request template (.github/pull_request_template.md) and a basic CONTRIBUTING.md guide. The template includes essential checklists for security, performance, and general testing concerns to ensure a comprehensive review process prior to merging code into the repository.

It aims to prevent issues from slipping through the cracks, specifically regarding data storage, token handling, input validation, and rendering performance on React Native (useCallback, FlatList optimization). It also mandates checking whether an Architectural Decision Record (ADR) is present for architectural changes.

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)
  • Chore / Refactor (no functional changes)

Checklist

  • Added .github/pull_request_template.md with standard sections (Summary, Type of Change, Testing Done, Checklist).
  • Added Security checklist (sensitive data storage, token exposure, input validation, deep linking).
  • Added Performance checklist (memo/useCallback, FlatList optimizations, bundle size impacts).
  • Added ADR checkbox for structural and architectural changes.
  • Created CONTRIBUTING.md referencing the new template and onboarding guidelines.

@drips-wave

drips-wave Bot commented Jun 27, 2026

Copy link
Copy Markdown

@AugistineCreates Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@RUKAYAT-CODER

Copy link
Copy Markdown
Contributor

Thank you for contributing to the project.

@RUKAYAT-CODER RUKAYAT-CODER merged commit 981c306 into rinafcode:main Jun 27, 2026
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.

[Enhancement] Add PR template with checklist including security and performance review items

2 participants