Skip to content

chore: improve PR template#171

Merged
i-am-SR merged 2 commits intomainfrom
prtemp
Feb 24, 2026
Merged

chore: improve PR template#171
i-am-SR merged 2 commits intomainfrom
prtemp

Conversation

@i-am-SR
Copy link
Contributor

@i-am-SR i-am-SR commented Feb 24, 2026

Description

Why is this change being made?

  1. Standardize the PR process by adding reviewee and reviewer checklists that ensure consistent validation steps, improve code quality, and make reviews more efficient.

What is changing?

  1. Setting up change review template for the repo.

Related Links

  • Issue #, if available: N/A

Testing

How was this tested?

  1. Validated the MD preview. The template needs to be merged to the main branch to be auto applied.

When testing locally, provide testing artifact(s):

  1. N/A

Reviewee Checklist

Update the checklist after submitting the PR

  • I have reviewed, tested and understand all changes
    If not, why:
  • I have filled out the Description and Testing sections above
    If not, why:
  • Build and Unit tests are passing
    If not, why: Just an MD update. No code changes.
  • Unit test coverage check is passing
    If not, why: Just an MD update. No code changes.
  • Integration tests pass locally
    If not, why: Just an MD update. No code changes.
  • I have updated integration tests (if needed)
    If not, why: Just an MD update. No code changes.
  • I have ensured no sensitive information is leaking (i.e., no logging of sensitive fields, or otherwise)
    If not, why:
  • I have added explanatory comments for complex logic, new classes/methods and new tests
    If not, why: Just an MD update. No code changes.
  • I have updated README/documentation (if needed)
    If not, why: No relevant doc changes required.
  • I have clearly called out breaking changes (if any)
    If not, why: Just an MD update. No code changes.

Reviewer Checklist

All reviewers please ensure the following are true before reviewing:

  • Reviewee checklist has been accurately filled out
  • Code changes align with stated purpose in description
  • Test coverage adequately validates the changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@i-am-SR i-am-SR requested a review from a team as a code owner February 24, 2026 00:38
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.14%. Comparing base (b2a5dd0) to head (4e078df).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #171   +/-   ##
=======================================
  Coverage   92.14%   92.14%           
=======================================
  Files          14       14           
  Lines        2405     2405           
  Branches     2405     2405           
=======================================
  Hits         2216     2216           
  Misses        143      143           
  Partials       46       46           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@i-am-SR i-am-SR added the safe-to-test Maintainer approval to run integration tests for external contributor PRs. label Feb 24, 2026
@github-actions github-actions bot removed the safe-to-test Maintainer approval to run integration tests for external contributor PRs. label Feb 24, 2026
@i-am-SR i-am-SR merged commit cbd8fd6 into main Feb 24, 2026
12 checks passed
@i-am-SR i-am-SR deleted the prtemp branch February 24, 2026 18:27
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.

3 participants