Skip to content

Commit

Permalink
Update PR Template for better Phabricator parsing
Browse files Browse the repository at this point in the history
Summary: Pull Request resolved: pytorch#2916

Differential Revision: D15349297

Pulled By: jfix71

fbshipit-source-id: 460a7a17f32fd7fad01ca77453d7d8ee0773e252
  • Loading branch information
jfix71 authored and facebook-github-bot committed May 15, 2019
1 parent 902bc9a commit 0239ff3
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 8 deletions.
9 changes: 4 additions & 5 deletions PULL_REQUEST.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,19 +9,19 @@ For example:

Ideally, the same format should be used in the commit message.

# Description
# Summary

The description field should include a more verbose explanation of what this PR does.
The Summary field should include a more verbose explanation of what this PR does.
If this PR causes a change in behavior it should document the behavior before and after.
If it fixes a bug, please, describe what the original issue is and how the change resolves it.

# Formatting

Please, run clang-format before submitting the PR.

# Testing
# Test Plan

The testing section should include an explanation of what testing was done.
The test plan section should include an explanation of what testing was done.
For example unit test, manual testing through stand-alone testing binaries like mnist, cifar10, etc.
Please use your best judgment or ask for guidance if you are unsure what kind of testing is required for a given change.
The riskier the change, the more comprehensive the testing should be.
Expand All @@ -41,4 +41,3 @@ This way Github will make sure that underlying referenced issue will be automati

If the purpose of the PR to address issue partially, please, just tag the issue:
* #Issue.

9 changes: 6 additions & 3 deletions PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
*Description*:
*Testing*:
*Documentation*:
Summary:

Documentation:

Test Plan:

[Optional Fixes #issue]

Please see a detailed explanation of how to fill out the fields in the relevant sections in PULL_REQUEST.md.

0 comments on commit 0239ff3

Please sign in to comment.