Skip to content

Commit c0a4f9a

Browse files
VLanvinfacebook-github-bot
authored andcommitted
Set clause coverage to true by default
Summary: As title, update docs accordingly. Leaving the `--clause-coverage` option in for now (it doesn't do anything) as it will be useful for coverage of non-top-level clauses. Can be overridden by `EQWALIZER_CLAUSE_COVERAGE=false`. Reviewed By: michalmuskala Differential Revision: D59574467 fbshipit-source-id: 0ba77be16b81b29d1b7edc331a4da9e2e2fe4696
1 parent 67fe4fe commit c0a4f9a

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

docs/reference/advanced.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -175,6 +175,6 @@ attempt to detect and report redundant type assertions. See [redundant_guard err
175175

176176
### Spec coverage of function clauses
177177

178-
Enable checks of proper coverage of function clauses by specs using `EQWALIZER_CLAUSE_COVERAGE=true`.
179-
By default, eqWAlizer will not check coverage of function clauses by the corresponding spec,
180-
hence a clause may be implicitly left unchecked.
178+
Disable checks of proper coverage of function clauses by specs using `EQWALIZER_CLAUSE_COVERAGE=false`.
179+
By default, eqWAlizer will check coverage of function clauses by the corresponding spec,
180+
ensuring all clauses are properly checked.

eqwalizer/src/main/resources/application.conf

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,6 @@ eqwalizer {
1717
mode = ${?EQWALIZER_MODE}
1818
error_depth = 4
1919
error_depth = ${?EQWALIZER_ERROR_DEPTH}
20-
clause_coverage = false
20+
clause_coverage = true
2121
clause_coverage = ${?EQWALIZER_CLAUSE_COVERAGE}
2222
}

0 commit comments

Comments
 (0)