From 81e66d7b042bcc2e56259d1b806877024043b6cd Mon Sep 17 00:00:00 2001
From: Barbz <BarbzYHOOL@users.noreply.github.com>
Date: Fri, 6 Jul 2018 22:15:44 +0200
Subject: [PATCH] Github/pull_request_template.md - Improved

Made "target branch" optional since all the PRs are for the master branch by default
Tidied the existing comments to be shorter to read but with more information
Added links for contribution
---
 pull_request_template.md | 65 ++++++++++++++++++++++------------------
 1 file changed, 36 insertions(+), 29 deletions(-)

diff --git a/pull_request_template.md b/pull_request_template.md
index 3995224c3..47097cf77 100644
--- a/pull_request_template.md
+++ b/pull_request_template.md
@@ -1,4 +1,16 @@
-<!-- First of all, thank you for your contribution. If you intend to contribute more than once, you should really join us on our discord channel! (link is on site, in the readme etc) We can set cosmetic ranks for contributors like you! -->
+<!-- First of all, THANK YOU for your contribution.
+<!-- ---------------------------------------------- -->
+<!-- Please fill this template unless your PR is very simple/straightforward.
+<!-- Do not forget to have a look at our Pull Request tutorial:
+<!-- https://github.com/azerothcore/azerothcore-wotlk/wiki/Contribute#how-to-create-a-pull-request
+<!-- ---------------------------------------------- -->
+<!-- If you intend to contribute more than once, you should really join us on our discord channel!
+<!-- The link is on our site http://azerothcore.org/ or in the readme https://github.com/azerothcore/azerothcore-wotlk etc...
+<!-- We set cosmetic ranks for our contributors and may give access to special resources/knowledge to them! -->
+
+
+<!-- WRITE A RELEVANT TITLE -->
+
 
 **Changes proposed:**
 
@@ -6,50 +18,45 @@
 -  
 
 
-**Target branch(es):**
-
-1.x/2.x etc.
-
-
 **Issues addressed:**
 
-<!-- Just paste the link to the issue you close -->
-Closes #
+<!-- Just paste the link to the issue you close, like this:
+<!-- Closes https://github.com/azerothcore/azerothcore-wotlk/issues/967 -->
+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)-->
+<!--  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?
+<!--  Other tests performed -->
 
 
 **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 -->
+<!-- Describe how to test the fix after applying the Pull Request. If needed describe how to test the issue before applying the <!-- Pull Request (for example, if the issue has not been reported before).
+<!--  - 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...
+<!--  - We need to confirm the changes first, so try to make the work easy for testers, please. -->
 
 
 **Known issues and TODO list:**
 
-<!-- This is a list with checkboxes -->
-- [ ] 
+<!-- This is a TODO list with checkboxes to tick -->
+- [ ]
 - [ ] 
 
 
-
-<!--
-/!\
-/!\
-**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).
-/!\
-/!\
--->
+<!-- OPTIONAL - If your Pull Request is not targeting the Master branch then remove the comments and specify the Target Branch 
+<!-- **Target branch(es):**
+---- 1.x/2.x etc. -->
+
+
+<!-- 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 for a cleaner and easy to read history). -->