Merge branch 'master' of https://github.com/BrentOzarULTD/SQL-Server-…

…First-Responder-Kit
BrentOzarULTD · Nov 29, 2018 · 89cdb34 · 89cdb34
2 parents 3490efd + aa880ec
commit 89cdb34
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 10 deletions.
diff --git a/.github/ISSUE_TEMPLATE.md → .github/bug_report.md b/.github/ISSUE_TEMPLATE.md → .github/bug_report.md
@@ -1,5 +1,11 @@
-**Do you want to request a *feature* or report a *bug*?**
-If you're reporting a bug, include the version number of the script you're using. If it's not the current version, upgrade to the current version and test that before reporting a bug - we fix a lot of stuff in each new build.
+---
+name: Bug report
+about: Create a bug report to help us improve
+
+---
+
+Include the version number of the script you're using.
+If it's not the current version, upgrade to the current version and test that before reporting a bug - we fix a lot of stuff in each new build.
 
 **What is the current behavior?**
 

diff --git a/.github/PULL_REQUEST_TEMPLATE.md → .github/feature_request.md b/.github/PULL_REQUEST_TEMPLATE.md → .github/feature_request.md
@@ -1,14 +1,16 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+
+---
+
 Fixes # .
 
 Changes proposed in this pull request:
  - 
- - 
- - 
 
 How to test this code:
  - 
- - 
- - 
 
 Has been tested on (remove any that don't apply):
  - Case-sensitive SQL Server instance
@@ -17,6 +19,7 @@ Has been tested on (remove any that don't apply):
  - SQL Server 2012
  - SQL Server 2014
  - SQL Server 2016
-  - SQL Server 2017
+ - SQL Server 2017
+ - SQL Server 2019
  - Amazon RDS
  - Azure SQL DB
diff --git a/Documentation/sp_BlitzCache Checks by Priority.md b/Documentation/sp_BlitzCache Checks by Priority.md
@@ -6,8 +6,8 @@ Before adding a new check, make sure to add a Github issue for it first, and hav
 
 If you want to change anything about a check - the priority, finding, URL, or ID - open a Github issue first. The relevant scripts have to be updated too.
 
-CURRENT HIGH CHECKID: 61
-If you want to add a new check, start at 62
+CURRENT HIGH CHECKID: 62
+If you want to add a new check, start at 63
 
 | Priority | FindingsGroup | Finding | URL | CheckID | Expert Mode | 
 |----------|---------------------------------|---------------------------------------|-------------------------------------------------|----------|-------------|
@@ -72,7 +72,7 @@ If you want to add a new check, start at 62
 | 100 | tempdb Spills | This query spills >500mb to tempdb on average | https://www.brentozar.com/blitzcache/tempdb-spills/ | 59 | No |
 | 100 | MSTVFs | These have many of the same problems scalar UDFs have | http://brentozar.com/blitzcache/tvf-join/ | 60 | No |
 | 100 | Many to Many Merge | These use secret worktables that could be doing lots of reads | Blog not published yet | 61 | Yes |
-| 50 | Non-SARGable queries | Queries may have non-SARGable predicates |http://brentozar.com/go/sargable| http://brentozar.com/go/sargable | No |
+| 50 | Non-SARGable queries | Queries may have non-SARGable predicates |http://brentozar.com/go/sargable| 62 | No |
 | 254 | Plan Cache Information | Breaks cache down by creation date (24/4/1 hrs) | None | 999 | No |
 | 255 | Global Trace Flags Enabled | You have Global Trace Flags enabled on your server | https://www.brentozar.com/blitz/trace-flags-enabled-globally/ | 1000 | No |
 | 255 | Need more help? | Paste your plan on the internet! | http://pastetheplan.com | 2147483646 | No |