Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
90 changes: 90 additions & 0 deletions .crux_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,90 @@
## Description

### Why is this change being made?

1.


### What is changing?

1.


### Related Links
- **Ticket/Issue**:
- **Design Doc**:

---

## Revision History

<!-- Create a new entry in this section when publishing new revisions. Leave blank for Revision 1. -->

### Revision X:

#### What has changed since the last revision?

1.

---

## Testing

### How was this tested?

1.


### When testing locally, provide testing artifact(s):

1.

---

## Reviewee Checklist

**Update the checklist on each revision:**

- [ ] I have reviewed, tested and understand all changes

*If not, why:*
- [ ] I have filled out the Description and Testing sections above

*If not, why:*
- [ ] I have updated the Revision History section (if publishing a new revision)

*If not, why:*
- [ ] Build and Unit tests are passing

*If not, why:*
- [ ] Unit test coverage for changes is adequate (> 80%)

*If not, why:*
- [ ] Integration tests pass locally

*If not, why:*
- [ ] I have updated integration tests (if needed)

*If not, why:*
- [ ] 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:*
- [ ] I have updated README/documentation (if needed)

*If not, why:*
- [ ] I have clearly called out breaking changes (if any)

*If not, why:*

---

## Reviewer Checklist

**All reviewers please ensure the following are true before reviewing each revision:**

- Reviewee checklist has been accurately filled out
- Code changes align with stated purpose in description
- Test coverage adequately validates the changes
65 changes: 63 additions & 2 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,67 @@
*Issue #, if available:*
## Description

*Description of changes:*
### Why is this change being made?

1.


### What is changing?

1.


### Related Links
- **Issue #, if available**:

---

## Testing

### How was this tested?

1.

### When testing locally, provide testing artifact(s):

1.

---

## 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:*
- [ ] Unit test coverage check is passing
*If not, why:*
- [ ] Integration tests pass locally
*If not, why:*
- [ ] I have updated integration tests (if needed)
*If not, why:*
- [ ] 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:*
- [ ] I have updated README/documentation (if needed)
*If not, why:*
- [ ] I have clearly called out breaking changes (if any)
*If not, why:*

---

## 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.
Loading