From 5bbd8f809a7765f231e1b5a0e4e3f653993a1c57 Mon Sep 17 00:00:00 2001
From: Mohamed Hegazy <mhegazy@microsoft.com>
Date: Wed, 15 Mar 2017 16:15:59 -0700
Subject: [PATCH] rename catogories, and reorder options

---
 src/compiler/commandLineParser.ts    | 48 ++++++++++++++--------------
 src/compiler/diagnosticMessages.json |  4 +--
 2 files changed, 26 insertions(+), 26 deletions(-)

diff --git a/src/compiler/commandLineParser.ts b/src/compiler/commandLineParser.ts
index f5cddf0e659..1e6a89d4440 100644
--- a/src/compiler/commandLineParser.ts
+++ b/src/compiler/commandLineParser.ts
@@ -238,68 +238,70 @@ namespace ts {
             description: Diagnostics.Unconditionally_emit_imports_for_unresolved_files
         },
 
-        // Strict Checks
+        // Strict Type Checks
         {
             name: "strict",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Strict_Type_Checks,
             description: Diagnostics.Enable_all_strict_type_checks
         },
         {
             name: "noImplicitAny",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Strict_Type_Checks,
             description: Diagnostics.Raise_error_on_expressions_and_declarations_with_an_implied_any_type,
         },
         {
             name: "strictNullChecks",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Strict_Type_Checks,
             description: Diagnostics.Enable_strict_null_checks
         },
         {
             name: "noImplicitThis",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Strict_Type_Checks,
             description: Diagnostics.Raise_error_on_this_expressions_with_an_implied_any_type,
         },
+        {
+            name: "alwaysStrict",
+            type: "boolean",
+            showInSimplifiedHelpView: true,
+            category: Diagnostics.Strict_Type_Checks,
+            description: Diagnostics.Parse_in_strict_mode_and_emit_use_strict_for_each_source_file
+        },
+
+        // Additional Checks
         {
             name: "noUnusedLocals",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Additional_Checks,
             description: Diagnostics.Report_errors_on_unused_locals,
         },
         {
             name: "noUnusedParameters",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Additional_Checks,
             description: Diagnostics.Report_errors_on_unused_parameters,
         },
-        {
-            name: "alwaysStrict",
-            type: "boolean",
-            showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
-            description: Diagnostics.Parse_in_strict_mode_and_emit_use_strict_for_each_source_file
-        },
         {
             name: "noImplicitReturns",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Additional_Checks,
             description: Diagnostics.Report_error_when_not_all_code_paths_in_function_return_a_value
         },
         {
             name: "noFallthroughCasesInSwitch",
             type: "boolean",
             showInSimplifiedHelpView: true,
-            category: Diagnostics.Strict_Checks,
+            category: Diagnostics.Additional_Checks,
             description: Diagnostics.Report_errors_for_fallthrough_cases_in_switch_statement
         },
 
@@ -404,14 +406,6 @@ namespace ts {
             description: Diagnostics.Emit_the_source_alongside_the_sourcemaps_within_a_single_file_requires_inlineSourceMap_or_sourceMap_to_be_set
         },
 
-        // JSX
-        {
-            name: "jsxFactory",
-            type: "string",
-            category: Diagnostics.JSX_Options,
-            description: Diagnostics.Specify_the_JSX_factory_function_to_use_when_targeting_react_JSX_emit_e_g_React_createElement_or_h
-        },
-
         // Experimental
         {
             name: "experimentalDecorators",
@@ -427,6 +421,12 @@ namespace ts {
         },
 
         // Advanced
+        {
+            name: "jsxFactory",
+            type: "string",
+            category: Diagnostics.Advanced_Options,
+            description: Diagnostics.Specify_the_JSX_factory_function_to_use_when_targeting_react_JSX_emit_e_g_React_createElement_or_h
+        },
         {
             name: "diagnostics",
             type: "boolean",
diff --git a/src/compiler/diagnosticMessages.json b/src/compiler/diagnosticMessages.json
index 6546408fdce..e833d77651b 100644
--- a/src/compiler/diagnosticMessages.json
+++ b/src/compiler/diagnosticMessages.json
@@ -3137,7 +3137,7 @@
         "category": "Message",
         "code": 6172
     },
-    "Strict Checks": {
+    "Strict Type Checks": {
         "category": "Message",
         "code": 6173
     },
@@ -3149,7 +3149,7 @@
         "category": "Message",
         "code": 6175
     },
-    "JSX Options": {
+    "Additional Checks": {
         "category": "Message",
         "code": 6176
     },