Skip to content

Commit 51895dd

Browse files
authored
Make transitive reporting false by default. (#610)
* Make transitive reporting false by default. * Update CHANGELOG.md
1 parent e071754 commit 51895dd

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

CHANGELOG.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414

1515
#### :rocket: New Feature
1616

17-
- 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
18-
This feature comes from a conversation with @jfmengels on how https://github.com/jfmengels/elm-review is designed.
17+
- 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 https://github.com/rescript-lang/rescript-vscode/pull/610
18+
This feature comes from a conversation with @jfmengels on how https://github.com/jfmengels/elm-review is designed.
1919

2020
#### :bug: Bug Fix
2121

analysis/reanalyze/src/RunConfig.ml

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ let runConfig =
1717
projectRoot = "";
1818
suppress = [];
1919
termination = false;
20-
transitive = true;
20+
transitive = false;
2121
unsuppress = [];
2222
}
2323

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
DCE src/Dce.res
2-
issues:2
2+
issues:1
33

0 commit comments

Comments
 (0)