From dd845a1b7b093d48167d7d1b2684471d0f56548a Mon Sep 17 00:00:00 2001
From: "Juan Picado @jotadeveloper" <juanpicado19@gmail.com>
Date: Sun, 25 Feb 2018 22:05:43 +0100
Subject: [PATCH] chore: update github templates

---
 .github/issue_template.md        | 55 +++++++++++++-------------------
 .github/pull_request_template.md | 36 ++++++++++-----------
 2 files changed, 40 insertions(+), 51 deletions(-)

diff --git a/.github/issue_template.md b/.github/issue_template.md
index 93b7316fa..422c1d8fd 100644
--- a/.github/issue_template.md
+++ b/.github/issue_template.md
@@ -1,41 +1,32 @@
+<!--
+First of all, please read this section
+
+https://github.com/verdaccio/verdaccio/blob/master/CONTRIBUTING.md#reporting-a-bug
+
+Some advices before file an issue
+  * Give a brief explanation of the issue, suggestion or feature to request.
+  * If the issue is a question, provide as much information you have available.
+  * How can I do in order to reproduce it? What environment?
+  * Define which version the issue happens and whether previous version the behaviour is correct.
+  * Provide your config file is really helpful. Please be aware to hide sensisive data (passwords, server IP, etc) before post.
+-->
+
 #### My reason:
-
-<!-- 
-	a brief explanation of the issue, suggestion, feature
--->
-
 #### Steps to reproduce:
-
-<!-- 
-	(if it applies)
-	how can I do in order to reproduce it? environment? 
--->
-####  App Version:	
-<!-- 
-      Define which version the issue happens and whether previous version the behaviour is correct
--->
-
-####  Config file:	
-<!-- 
-      Provide your config file might be really helpful. Please be aware to hide sensisive data before post.
--->
-
+#### App Version:
+#### Config file:
 #### Additional information:
 
-<!--
-	provide the following information would be helpful
--->
-	
- - `$ set DEBUG=express:* verdaccio` enable extreme verdaccio debug mode
- - `$ npm --verbose` prints:
+ - `$ NODE_DEBUG=request verdaccio` display request calls (verdaccio <--> uplinks)
+ - `$ DEBUG=express:* verdaccio` enable extreme verdaccio debug mode (verdaccio api)
+ - `$ npm -ddd` prints:
  - `$ npm config get registry` prints:
  - Verdaccio terminal output
- - Windows, OS X/macOS, or Linux?:
+ - Which (Windows, OS X/macOS, or Linux) environment are you running verdaccio?:
  - Verdaccio configuration file, eg: `cat ~/.config/verdaccio/config.yaml`
- <!-- Remove those does not apply for you -->
- - Container:
-   - I use local environment
-   - I develop / deploy using Docker.
-   - I deploy to a PaaS.
+  - Container Options:
+   - Docker?:
+   - Kubernetes?:
 
 #### Additional verbose log:
+
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
index 42b17bae5..4de3b73fb 100644
--- a/.github/pull_request_template.md
+++ b/.github/pull_request_template.md
@@ -1,29 +1,27 @@
-<!-- Before Pull Request check whether your commits follow this convention
+<!--
+
+Before Pull Request check whether your commits follow this convention
+
 https://github.com/verdaccio/verdaccio/blob/master/CONTRIBUTING.md#git-commit-guidelines
--->
 
-<!-- 
-  If your Pull Request fix an issue don't forget to update the unit test and documentation in /docs folder
-  If your Pull Request delivers a new feature, please, provide examples and why such feature should be considered, this information is important to document the Github changelog. Also, try to increase documentation and create new unit/functional test.  
--->
+  * If your PR fix an issue don't forget to update the unit test and documentation in /docs folder
+  * If your PR delivers a new feature, please, provide examples and why such feature should be considered.
+  * Document your changes /docs
+  * Add unit test
+  * Follow the commit guidelines in order to get a quick approval
 
-<!-- Pick only one type, whether none apply, please suggest one, we might be included it by default -->
-**Type:** bug / feature / documentation / unit test / build
+Pick one/multiple type, if none apply please suggest one, we might be included it by default
+
+eg: bug / feature / documentation / unit test / build
+
+-->
+**Type:**
 
 The following has been addressed in the PR:
 
-<!-- Remove the sections that your PR does not apply -->
-*  There is a related issue
+*  There is a related issue?
 *  Unit or Functional tests are included in the PR
 
-<!--
-Our bots should ensure:
-* The PR passes CI testing
--->
-
-<!-- If there is no issue related to this PR just remove the following section -->
 **Description:**
 
-Resolves #???
-
-<!-- We are glad your code is part of this community, thanks for your valuable time !! --> 
+<!-- Resolves #??? -->