Skip to content

chore: update PR template#190

Merged
i-am-SR merged 1 commit intov2from
prtemp
Mar 10, 2026
Merged

chore: update PR template#190
i-am-SR merged 1 commit intov2from
prtemp

Conversation

@i-am-SR
Copy link
Contributor

@i-am-SR i-am-SR commented Mar 5, 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.
  • 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 March 5, 2026 00:48
@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.90%. Comparing base (a256a25) to head (749b082).
⚠️ Report is 1 commits behind head on v2.

Additional details and impacted files
@@            Coverage Diff            @@
##                 v2     #190   +/-   ##
=========================================
  Coverage     90.90%   90.90%           
  Complexity       90       90           
=========================================
  Files             6        6           
  Lines           231      231           
  Branches         24       24           
=========================================
  Hits            210      210           
  Misses           18       18           
  Partials          3        3           

☔ 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 merged commit 6b428ef into v2 Mar 10, 2026
5 checks passed
@i-am-SR i-am-SR deleted the prtemp branch March 10, 2026 21:55
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