Skip to content

Update PR template to be concise with checklist - #73

Merged
rapids-bot[bot] merged 2 commits into
NVIDIA:branch-25.08from
ramakrishnap-nv:update_pr_template
Jun 5, 2025
Merged

Update PR template to be concise with checklist #73
rapids-bot[bot] merged 2 commits into
NVIDIA:branch-25.08from
ramakrishnap-nv:update_pr_template

Conversation

@ramakrishnap-nv

@ramakrishnap-nv ramakrishnap-nv commented Jun 5, 2025

Copy link
Copy Markdown
Collaborator

Description

This PR simplifies PR template to have 3 sections, "Description", "Issue" and "Checklist". The check list provides overview on tests and documentation with respect to changes that are part of this PR.

Issue

There is no issue created for this requirement.

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue
    • NA
  • Documentation
    • The documentation is up to date with these changes.
    • Added new documentation
    • NA

@ramakrishnap-nv
ramakrishnap-nv requested a review from a team as a code owner June 5, 2025 15:40
@ramakrishnap-nv
ramakrishnap-nv requested a review from bdice June 5, 2025 15:40
@ramakrishnap-nv ramakrishnap-nv self-assigned this Jun 5, 2025
@ramakrishnap-nv ramakrishnap-nv added doc Improvements or additions to documentation non-breaking Introduces a non-breaking change labels Jun 5, 2025

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Simpler checklists are more likely to be following, in my opinion.

In the future, you might even want to consider GitHub forms: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/syntax-for-issue-forms

Those are nice because they can make inputs mandatory.

@jameslamb
jameslamb removed the request for review from bdice June 5, 2025 15:58
@ramakrishnap-nv

Copy link
Copy Markdown
Collaborator Author

/merge

@rapids-bot
rapids-bot Bot merged commit ff21913 into NVIDIA:branch-25.08 Jun 5, 2025
jieyibi pushed a commit to yining043/cuopt that referenced this pull request Mar 26, 2026
This PR simplifies PR template to have 3 sections, "Description", "Issue" and  "Checklist". The check list provides overview on tests and documentation with respect to changes that are part of this PR.

## Issue
There is no issue created for this requirement.

Authors:
  - Ramakrishnap (https://github.com/rgsl888prabhu)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: NVIDIA#73
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Improvements or additions to documentation non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants