Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update PR template #823

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

docs: update PR template #823

wants to merge 1 commit into from

Conversation

bk958178
Copy link
Contributor

@bk958178 bk958178 commented Oct 25, 2024

PR Type

  • Documentation content changes

Description

This PR updates the template for future PRs, in light of inefficiencies and disengagement with certain pre-existing sections.

How Has This Been Tested?

Existing tests pass.

Does this PR introduce a breaking change?

No.

(Write your answer here.)

Checklist before requesting a review

  • I have made sure that my PR is not a duplicate.
  • My code follows the style guidelines of this project.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have performed a self-review of my code.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • Any dependent changes have been merged and published in downstream modules.

Copy link
Contributor

Performance review

Commit 2d3dfea - Merge 13abe04 into bc5acf0

No significant changes to performance.

Copy link
Contributor

@rg936672 rg936672 left a comment

Choose a reason for hiding this comment

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

Looks good. I'd suggest replacing the "I have commented my code" check in the checklist with something more like "I have ensured my code is easy to understand, including adding docstrings and using comments where necessary", since some changes just don't really require comments.

Test A: (Write your answer here.)
Test B: (Write your answer here.)
Test C: (Write your answer here.)
(Write your answer here, or delete as appropriate.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove these lines that need to be deleted by the user?

@bk958178 bk958178 linked an issue Oct 28, 2024 that may be closed by this pull request
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.

Update PR template to improve efficiency
3 participants