Skip to content

Commit

Permalink
Merge pull request #40 from Financial-Times/fix/pr-template-7955
Browse files Browse the repository at this point in the history
Updating Pull Request Template.
  • Loading branch information
georgi-denchew committed Jun 9, 2020
2 parents d98e3ed + acd7a23 commit 7dd6501
Showing 1 changed file with 29 additions and 0 deletions.
29 changes: 29 additions & 0 deletions pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
# Description

## What

Please be specific and try to describe your thought process. State the obvious, since this might be the first time the reviewer is looking at the code

## Why

Copy (if there is one) the text of the original Trello/JIRA ticket in here, with a link back to it for the curious.

## Anything, in particular, you'd like to highlight to reviewers

Mention here sections of code which you would like reviewers to pay extra attention to .E.g

_Would appreciate a second pair of eyes on the test_
_I am not quite sure how this bit works_
_Is there a better library for doing x_

## Scope and particulars of this PR (Please tick all that apply)

- [ ] Tech hygiene (dependency updating & other tech debt)
- [ ] Bug fix
- [ ] Feature
- [ ] Documentation
- [ ] Breaking change
- [ ] Minor change (e.g. fixing a typo, adding config)

___
This Pull Request follows the rules described in our [Pull Requests Guide](https://github.com/Financial-Times/upp-docs/tree/master/guides/pr-guide)

0 comments on commit 7dd6501

Please sign in to comment.