Make transitive reporting false by default.

CHANGELOG.md

@@ -14,7 +14,7 @@
 
 #### :rocket: New Feature
 
-- Add configuration parameter `"transitive"` under `"reanalyze"` is `bsconfig.json`. If set to `false`, the analysis does not report transitively dead items. So removing the reported item individually can be done in isolation. This is a more fine-grained process for guiding the user to remove dead code one item at a time. https://github.com/rescript-lang/rescript-vscode/pull/601
+- Add configuration parameter `"transitive"` under `"reanalyze"` is `bsconfig.json` and make reportst non-transitive by default. If set to `false`, the analysis does not report transitively dead items. So removing the reported item individually can be done in isolation. This is a more fine-grained process for guiding the user to remove dead code one item at a time. https://github.com/rescript-lang/rescript-vscode/pull/601
 This feature comes from a conversation with @jfmengels on how https://github.com/jfmengels/elm-review is designed.
 
 #### :bug: Bug Fix

analysis/reanalyze/src/RunConfig.ml

@@ -17,7 +17,7 @@ let runConfig =
     projectRoot = "";
     suppress = [];
     termination = false;
-    transitive = true;
+    transitive = false;
     unsuppress = [];
   }
 

analysis/tests/src/expected/Dce.res.txt

@@ -1,3 +1,3 @@
 DCE src/Dce.res
-issues:2
+issues:1