diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 2672715df..563df3116 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -4,7 +4,7 @@ This section can be deleted after reading. We employ the following branching strategy to simplify the development process and to ensure that only stable code is pushed to the `master` branch: - `develop`: For unstable code: New features and bug fixes. -- `master`: Where the stable production ready code lies. Only security related bugs. +- `master`: Where the stable production-ready code lies. Only security-related bugs. NOTE!!! @@ -15,39 +15,46 @@ ONLY SUBMIT PRS AGAINST OUR `DEVELOP` BRANCH. THE DEFAULT IS `MAIN`, SO YOU WILL Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. --> -**What kind of change does this PR introduce?** +### What kind of change does this PR introduce? -**Issue Number:** +### Issue Number: Fixes # -**Did you add tests for your changes?** +### Did you add tests for your changes? -**Snapshots/Videos:** +- [ ] Tests are written for all changes made in this PR. +- [ ] Test coverage meets or exceeds the current coverage (~90/95%). + +Snapshots/Videos: -**If relevant, did you update the documentation?** +### If relevant, did you update the documentation? -**Summary** +### Summary - + -**Does this PR introduce a breaking change?** +### Does this PR introduce a breaking change? -**Other information** +### Checklist for Repository Standards +- [ ] Have you reviewed and implemented all applicable `coderaabbitai` review suggestions? +- [ ] Have you ensured that the PR aligns with the repository’s contribution guidelines? + +### Other information -**Have you read the [contributing guide](https://github.com/PalisadoesFoundation/talawa/blob/master/CONTRIBUTING.md)?** +### Have you read the [contributing guide](https://github.com/PalisadoesFoundation/talawa/blob/master/CONTRIBUTING.md)? diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index cd3339781..4abd4d8d0 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -250,6 +250,8 @@ The value of `safeBlockVertical` and `safeBlockHorizontal` will be displayed in - All your file should contain at max `300` lines of code. - Follow proper code formatting and run `flutter format .` before your PR. - Run `flutter analyze` before your PR and make sure to resolve all the found issues. + - Tests must be written for all code changes, ensuring they maintain or exceed the current code coverage threshold. + - Ensure your pull request is approved by Coderaabbitai by addressing and implementing all its suggestions. ### Project Structure