Skip to content
This repository has been archived by the owner on Feb 28, 2024. It is now read-only.

Latest commit

 

History

History
32 lines (20 loc) · 1.45 KB

pull_request_guide.md

File metadata and controls

32 lines (20 loc) · 1.45 KB

Pull Request(PR) Guide, Resources and Checklists

PR Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. Additionally, adding types of changes can be a helpful descriptor.

Types of changes
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Links

Please provide links to related issues and PRs. Also, it may be relevant to provide links to resources and documentation used. Example: "I used the MDN Wed Docs" on the date index...."

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Helpful Checklist:

  • I have performed a self-review of my own code
  • I've reviewed and removed logs and testing variables
  • I have commented my code in particularly hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Incomplete PRs.

  • Apply a [wip] label for other developers to help and review.