fixes #74, introduces LR0004 WARNING diagnostic on unused facts (not …

[bkaptijn]

…referenced)

[codecov-io]

Codecov Report

Merging #82 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #82   +/-   ##
=======================================
  Coverage   96.08%   96.08%           
=======================================
  Files           3        3           
  Lines         511      511           
=======================================
  Hits          491      491           
  Misses         20       20

Impacted Files	Coverage Δ
src/modelValidator.js	94.47% <100%> (+0.32%)	⬆️
src/util.js	96.29% <0%> (-0.93%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fa2e27...5c0df84. Read the comment docs.

[pimotte]

LGTM, just one small nitpick:)

Can you also bump the patch version of the package before merging? (Also needs a back-merge from master due to merge conflicts.)

[pimotte]

This kind of duplicates the information in the 'source' attribute. In the end in vscode this would show as something like: "Unused fact [unusedfact] ([unusedfact])", so I would remove the fact.fact in the message, and just have 'Unused fact'