From 825a62b8d3d2679eeb782b7685bc6c6ceb6f56f1 Mon Sep 17 00:00:00 2001
From: Barbz <BarbzYHOOL@users.noreply.github.com>
Date: Wed, 23 May 2018 15:53:12 +0200
Subject: [PATCH] Improve the pull request template (#882)

* Improve the pull request template

My take on making the pull request template more useful

* Improvement - Which OS did you test on?

Added "which OS did you test on"
---
 pull_request_template.md | 43 +++++++++++++++++++++++++++++++++++-----
 1 file changed, 38 insertions(+), 5 deletions(-)

diff --git a/pull_request_template.md b/pull_request_template.md
index 06c7662c2..1ad27c070 100644
--- a/pull_request_template.md
+++ b/pull_request_template.md
@@ -1,20 +1,53 @@
 **Changes proposed:**
 
--  
 -  
 -  
 
-**Target branch(es):** 1.x/2.x etc.
 
-**Issues addressed:** Closes #
+**Target branch(es):**
+
+1.x/2.x etc.
+
+
+**Issues addressed:**
+
+<!-- Just paste the link to the issue you close -->
+Closes #
+
+
+**Tests performed:**
+
+<!-- Does it build without errors?
+Did you test in-game?
+What did you test?
+Did you do all these tests on Linux, Mac or Windows?
+etc)-->
+
+
+**How to test the changes:**
+
+<!--
+Describe how to test the issue before and after the pull request.
+Which commands to use? Which NPC to teleport to?
+Do we need to have debug flags on Cmake?
+Do we need to look at the console? etc...
+Try to make the work easy for testers, please -->
 
-**Tests performed:** (Does it build, tested in-game, etc)
 
 **Known issues and TODO list:**
 
+<!-- This is a list with checkboxes -->
 - [ ] 
 - [ ] 
 
-**NOTE** You no longer need to squash your commits, on merge we will squash it for you. (GitHub added a new feature)
+
+
+<!--
+/!\
+/!\
+**NOTE** You no longer need to squash your commits, on merge we will squash them for you (when there are too many commits we merge them into one big commit).
+/!\
+/!\
+-->