node-elm-review

Now requires jfmengels/elm-review v2.14.0.

Supports analyzing arbitrary files that the project doesn't analyze by default.

• The --ignore-dirs and --ignore-files flags now support absolute paths. Thanks @jamesrweb!
• Fixed an issue where the CLI output would sometimes be cut off abruptly. Thanks @BrianHicks!
• When using --watch and --report=json, reviews will now be triggered when files are changed even when the contents are triggered, to make it easier for editors to integrate elm-review.

https://github.com/jfmengels/node-elm-review/compare/v2.11.1...v2.11.2

• Fix crash when caching results in watch mode 956b2ccd
• Improve error message when failing to update a suppression file 0c82b00c

https://github.com/jfmengels/node-elm-review/compare/v2.11.0...v2.11.1

• Add an offline mode to prevent elm-review from making any HTTP requests. This is useful for CI environments that should not have access to the internet, where you only want to run elm-review without arguments.
• Introduce prepare-offline subcommand to have elm-review fetch and pre-build all the necessary data when it has network access to prepare for being run using elm-review --offline.
• Replaced the internally used elm-json dependency with elm-solve-deps-wasm, which should be more reliable, re-enable support for old MacOS versions as well as improve performance.
• Fixed a crash when using --template from the same repository but from two different paths.
• Fixed an issue where project rules created using elm-review new-rule mixed the ModuleContext and ProjectContext.

https://github.com/jfmengels/node-elm-review/compare/v2.10.0...v2.11.0

• Update snapshots 5abe389f
• Remove test dependencies when compiling review/elm.json b90c446a

https://github.com/jfmengels/node-elm-review/compare/v2.11.0-beta.2...v2.11.0-beta.3

• Fix illegal FS operation on versions cache file c91aa085

https://github.com/jfmengels/node-elm-review/compare/v2.11.0-beta.1...v2.11.0-beta.2

• Add an offline mode to prevent elm-review from making any HTTP requests. This is useful for CI environments that should not have access to the internet, where you only want to run elm-review without arguments.
• Replaced the internally used elm-json dependency with elm-solve-deps-wasm, which should be more reliable, re-enable support for old MacOS versions as well as improve performance.
• Fixed a crash when using --template from the same repository but from two different paths.

This is a BETA release, so I expect things to break. Please report any issues you encounter on the #elm-review channel on the Elm Slack.

https://github.com/jfmengels/node-elm-review/compare/v2.10.3...v2.11.0-beta.1

• Fixed crash when too many errors were reported.
• Improved guidance after using elm-review init --template <some-template>.

https://github.com/jfmengels/node-elm-review/compare/v2.10.2...v2.10.3

• Bump http-cache-semantics from 4.1.0 to 4.1.1 c6066084
• Bump json5 and tsconfig-paths 72ae555a
• Bump decode-uri-component from 0.2.0 to 0.2.2 f256efc3
• Bump minimatch from 3.0.4 to 3.0.8 248eb108
• Remove mentions of 'Downloading elm-json' in test output f8fdbe73
• Fix formatting e538ad19
• Add a hint in the prompt 479d85c9
• Generate a separate template for project rules a2b790ec
• Split body template into a separate function 4bba8a42
• Add the --rule-type flag to new-package 42a832d8
• Prompt for the rule-type if needed 4d0cf4ff
• Add --rule-type in tests and create a project rule in the tests 691fa8d0
• Add --rule-type flag to new-rule 731f5152
• Add a test by default that does not expect errors d8fe9984
• Change the template for new-rule 86e7dd06
• Update dependencies created by new-package 58e1ebfb
• Fix formatting issues dd928fbc
• Fix file caching issue on Windows a8d37e7b
• Use longer timeout in jest 7fbe0581
• Fix xo errors 0f472bfe
• Cache replacer: Don't crash when null is encountered d35e8211
• Update snapshots a0f3095c
• Enable NoDeprecated in the review configuration for new-package 983c073e
• Bump dependencies for new-package review configuration d1fe9b9d

https://github.com/jfmengels/node-elm-review/compare/v2.9.1...v2.9.2

What's Changed

• Fix CI failures by @kachick in https://github.com/jfmengels/node-elm-review/pull/104
• Fix watch issue on Windows by @bdukes in https://github.com/jfmengels/node-elm-review/pull/107
• Add a template for project rules using new-rule by @jfmengels in https://github.com/jfmengels/node-elm-review/pull/108
• Remove mentions of 'Downloading elm-json' in test output by @jfmengels in https://github.com/jfmengels/node-elm-review/pull/109
• Bump minimatch from 3.0.4 to 3.0.8 by @dependabot in https://github.com/jfmengels/node-elm-review/pull/92
• Bump decode-uri-component from 0.2.0 to 0.2.2 by @dependabot in https://github.com/jfmengels/node-elm-review/pull/88
• Bump json5 and tsconfig-paths by @dependabot in https://github.com/jfmengels/node-elm-review/pull/96
• Bump http-cache-semantics from 4.1.0 to 4.1.1 by @dependabot in https://github.com/jfmengels/node-elm-review/pull/102

New Contributors

• @kachick made their first contribution in https://github.com/jfmengels/node-elm-review/pull/104
• @bdukes made their first contribution in https://github.com/jfmengels/node-elm-review/pull/107

Full Changelog: https://github.com/jfmengels/node-elm-review/compare/v2.9.1...v2.9.2

Optimizes the internal stringification of project contexts. Hopefully this will improve performance, but it's mostly useful to avoid stack overflows during that process.

• Add changelog for 2.9.1 c1ee1097
• Shorten the serialization of lists 82f5bfb7
• Use the elmJsonReplacer to serialize in the Elm code 28a1a115

https://github.com/jfmengels/node-elm-review/compare/v2.9.0...v2.9.1

Now requires jfmengels/elm-review v2.11.0.

• Adds a file-system cache for the linting results, which improves performance by quite a bit.
• Improve the error message when indicated that the package version is not supported
• Fix a problem where in watch and fix mode, the CLI sometimes remained stuck.

• Add changelog for 2.9.0 b460890c
• Improve error message for outdated messages 8423b766
• Fix expected test output 0c93e9ab
• Prevent crashing when caching is disabled 7d34248f
• Require jfmengels/elm-review 2.11.0 or higher 3e6ef02d
• Upgrade elm-review dependency 4bac96ea
• Fix crash in --watch mode 1312fbee
• Make the appHash for a template its commit hash ad503b47
• Add JSON replacer/reviver to support NaN and Infinity values b08b5273
• Check if there is a promise to resolve before calling it 2011ba14
• Support injecting the cache when not using LOCAL_ELM_REVIEW_SRC 97a2fc3d
• Disable caching when files/dirs are specified or ignored 914d077c
• Fix crash resultCachePath in watch mode 8bbc0599
• Load the result cache when initializing the app 6fd33869
• Compute the cache folder inside ResultCache.load ab1ac952
• Disable result-caching in debug mode fc9589b2
• Make result-cache work for Node.js v10 7f7a0527
• Formatting 3c221c3f
• Make elmReviewResultCache a Map be096587
• Make promisesToResolve a Map 72d6a027
• Clear up the cache when re-loading the cache f212c1bb
• CLean up 621148c0
• Avoid recreating worker unnecessarily edbe0faf
• Load the result cache in watch mode as well 9e8e0b5b
• Write the result-cache files from a worker thread f85a6fd7
• Avoid creating the cache folder multiple times 37e6e184
• Show how long it took to write a result-cache file 845bbf7a
• Remove redundant error handling for caching results 7fc6cfc9
• Avoid crashing on invalid JSON 98bfd1ac
• Pass the cache key 9048f880
• Don't benchmark caching results ce13a513
• Remove unnecessary argument 742fb7e7
• Avoid rules with the same name from conflicting 0aa0ef38
• Provide a function rather than a record 4a519468
• Make sure that the cache is loaded before starting the review in the worker 9a7e2f58
• Load the cache in the app in the worker 53bef1b0
• Pass cache folder to ResultCache.load 9adb766f
• Add resultCacheFolder to the app flags 67114540
• Rename options.rules to options.rulesFilter ca8a1bc8
• Avoid writing to the memory result cache 8a6d7c8c
• Fix benchmarking c7979dea
• Save result cache for each rule independently when we're done reviewing it 487cd32b
• Formatting 88f25efd
• Fix waiting working on undefined 697962a9
• Using benchmark to log time spent caching results 5a032ef1
• Enable hacky communication for non-TTY systems dfc251b1
• Reformat code 78bfb9ff
• Don't apply caching replacements to the parser app build ce602cb0
• Remove try catch 42787fa1
• Define result cache location in options f80e783b
• Clean up result cache folder handling bd4c90b4
• Remove gzip compression 886a5c5e
• Remove exposing of safeConsoleLog 62971bed
• Benchmark printing report fceeee61
• Only load some rules if --rules is used 81676872
• Rename RuleCaches to ResultCache 748aff0d
• Cache stringifying a context 01b43e50
• Save cache prettified if in the debug mode c8b56d4e
• Update optimization targets f9928647
• Put hashes into a map 7b2e301e
• Don't try to load files if dir is empty 1eb4e0cb
• Add replacement to convert contexts to comparable integers 5a361f77
• Revert "DEBUG" 43ec50fe
• DEBUG d5bca60c
• Read individual cached files 26c4e478
• Fix gzip loading 1d46ee3e
• Only save individual files 713619ea
• Write rules individually c9c37223
• Remove hardcoding of appHash 2f8aca0d
• Rename RuleCaches functions 03b847bb
• Make reading async 3d4acfe0
• Move loading the cache at the same time as loading the files a5d27639
• Load cache in parallel of building the wrapper 1a5d54fc
• Exit earlier if there is no elm module path 61ee8025
• formatting 78be302e
• Add timers c3b6d6cd
• Make it easy to try out both gzip and simple json cd80d364
• Reorder arguments d67cd5b5
• Optimize the JS bundle even in debug mode dc3c7f8d
• TODO 47935a17
• Deflate the gzipped cache 695653f5
• Rename 5f58b937
• Succeed in gzipping the cache 3f7c2f9c
• Succeed in writing to the file system bf91ac00
• Write cache to file system 644cca80
• Read the cache 89a8cd3a
• Add mechanism to inject a cache 8fc9970a
• Update snapshots b50c04d5
• Prevent optimizing the code if the build failed 8d748906
• Fix watch mode blocking forever when undoing a fix fa4aea0b
• Update snapshots 7cda67e2

https://github.com/jfmengels/node-elm-review/compare/v2.8.5...v2.9.0

• Avoid saving the project in the initial run 09cd068
• Security vulnerability - Upgrade minimist to 1.2.6 (#89) 73d16c8
• Update snapshots a0dd88c

https://github.com/jfmengels/node-elm-review/compare/v2.8.4...v2.8.5

• Update snapshots bae3f0b
• Prepare changelog for 2.8.4 212b66e
• Add missing changelog 49a5524
• Fix incorrect reports in watch mode 55faca9
• Remove unnecessary Project.precomputeModuleGraph 9876dd7

https://github.com/jfmengels/node-elm-review/compare/v2.8.3...v2.8.4

• Don't call process.stdout.clearLine when the environment doesn't support it fba84c0
• Update snapshots 8aa63d0
• Document --extract flag 830e704
• Document --benchmark-info 95f4b26
• Update description for --debug in help 07c7bb6
• Give a more helpful error when using an outdated version of the package 7f0538b
• Update elm-test and elm-syntax in new packages 92c1477

https://github.com/jfmengels/node-elm-review/compare/v2.8.2...v2.8.3

Bug fix: the tests/ folder at the root of the project was not included in the folders to analyze

• Re-introduce looking for the tests/ folder at the root a3d3a2e
• Update initial configuration for new-package 9f693e2

https://github.com/jfmengels/node-elm-review/compare/v2.8.1...v2.8.2

(A technical mishap caused 2.8.0 to not be publishable, hence why the jump from 2.7.6 to 2.8.1)

Now requires running jfmengels/elm-review v2.10.0.

This release is partially announced by this announcement blog post.

• By using the package's new way of handling fixes, fixes have become much faster
• Introduce --fix-limit=N flag to limit the number of fixes to apply before prompting the user
• Introduce --extract to extract data from the project (needs to be run with --report=json)
• Include the information of how long a rule ran for when running with --benchmark-info
• More tests folders are now included. Before, elm-review looked at the source-directories plus the tests/ folder in the root of the Elm project.
  Instead, elm-review will now look for a tests/ folder next to every source-directory. This should accommodate projects with multiple Elm subprojects.

Commits: https://github.com/jfmengels/node-elm-review/compare/v2.7.6...v2.8.1

• Update changelog 4a7f5e0
• Update snapshots feb2c18
• Require jfmengels/elm-review v2.10.0 or higher 47a0ebf
• Fix crash when handling error 24ba303
• Ignore test-dependencies in elm.json when downloading using --template 5ab1e8a
• Document the extracts in tooling-integration 0e34fdb
• Don't enable computing extracts by default 0d61ec2
• Add missing changelog for 2.7.6 bea2388
• Report an error if fix-limit is not a positive integer 4882b94
• Change order of checks 44bef6e
• Always report extracts 557ab4b
• Add TODOs 9356cc7
• Compute cache of sources once 71977e8
• Have more consistent error handling for spawnElmJsonAsync 51e48af
• Document the --extract flag more 09c1400
• Fix extra lines printed out when fixing but not using --benchmark-info 8b5fecb
• Remove handler of rule-start and rule-end 7af4622
• Make CliCommunication.send take a list of properties 284164f
• Include tests that are stored in sub-folders 126529c
• Separate the line that count the number of fixes and the benchmark results 50c164e
• Add more inclusive terms e8e35c9
• Rename fix-logger to elm-communication 9fba028
• Remove clearFixProgress in favor of clearing inside the CLI db3f580
• Document Cli communication 4fc1901
• Rename Progress to CliCommunication d2e9040
• Remove unused message handling a9ff47e
• Use more pipelines 323c451
• Extract variable e958e1b
• Move functions to communicate with the CLI to Progress af69f0b
• Remove stray logging 78256bb
• Make timer easier and remove issue about label already existing 1217385
• Always use the apply-fix logging a362df6
• Specify fixes to ignore dbae5a9
• Put in AwaitingError mode 948b29b
• Make fixAllErrors a dict of ReviewError 46eceac
• Remove unnecessary assignment ead7c7b
• Re-run review in single fix mode 8b6d9b5
• Rename fixAll to applyFixesAfterReview 37e4e4d
• Return the new model in fixAll 456a6a4
• Rename field argument 2b6f7ad
• Remove unnecessary List.reverse 8fd1d3b
• Allow increasing the fix limit for --fix 849ef3a
• If using --fix-all always print the fix-all prompt 7859fed
• Extract function 4cc2bc0
• If there is a single error, then print the single fix proposal aad6950
• Add the file path 81588d6
• Remove unnecessary sending of error message 34d8698
• Remove constant argument to Reporter.formatFixProposal fe64412
• Add numberOfErrors2 cba99ba
• Extract variable e7b90e8
• Extract function 5247fcb
• Cause Mode_Fix to trigger the same process as fixAll fa6bcaa
• Mark argument as unused d76e02c
• Make fix limit nullable 5a733fe
• Code reformatting 01d35c8
• Add --fix-limit=N option 074d491
• Use ReviewOptions.withFixes 392b5c0
• Print the count of fixed errors from inside the message f42b5c0
• Show the number of errors in the fix prompt 8a8f8e6
• Store the rules with caching specific to the fix-all results only when changes have been accepted f609b42
• Prevent running fixes again when refusing fix-all 0f263fb
• After a fix has been applied, continue from the fixed project aa949d7
• Replace functions in compiled JS by faster ones 90595c2
• Fix infinite loop because we're applying fixes on the old model 821f3f6
• Add suppressed errors to review options b3e7936
• Register the fixed errors bd365aa
• Save result project under fixAllResultProject 723dc0b
• Print number of fixed errors 725b40c
• Print file path when we're fixing a file a81fd04
• Add fix all flag 38d156c
• Remove ProjectData and add Project 6bc86a2
• Fix report e489b92
• Only show rule performance when running with --benchmark-info 5cc2f74
• Add more logs 8e8f0a2
• Add more logs c7b582d
• Log fixes 7fe52bd
• Add timer a5ec1c5
• Connect a generic logger 5d7babe
• Add finishedRunningRule fdfe225
• Extract function 0c1ec75
• Pass objects as logger messages instead of constructed strings 3724830
• Use ReviewOptions e22d438
• Only extract when in report mode a25901c
• Properly print the entire extracts before exiting 6d122bb
• Print correct JSON 5127576
• Add extract to JSON output ff34de3
• WIP 0f2d3ea
• Send extracts to JavaScript d622d76
• Add extract option f047a8f
• Collect extracts 17c7ea6

• Update new-package to use elm-test v2 f605f94
• Update elm-test in the project 61c60b4
• Update snapshots 31975d2
• Add NoConfusingPrefixOperator rule to new-package configurations 162afa2
• Update snapshots 3d9cc70
• Update changelog 9f3ec46

https://github.com/jfmengels/node-elm-review/compare/v2.7.5...v2.7.6

• Fix crash during init --template de70b0a
• Update GitHub Actions workflow for packages 8dc40a9
• Bump elm-review packages for dependencies e9cd5c6
• Bump github actions dependencies ebf4a63

https://github.com/jfmengels/node-elm-review/compare/v2.7.4...v2.7.5

• Add changelog 32a1afa
• Change error message for remaining suppressed errors 6331b22
• Fix suppression errors problems on Windows (#80) c711aba
• Update caniuse 98c561f
• Fix formatting 43fa72e
• Report nicer error messages when a source-directory is unknown 39c8abf
• Update snapshots 75da80e
• Disables colors in tests 884afe0
• Force colors on by default 3d264da

https://github.com/jfmengels/node-elm-review/compare/v2.7.3...v2.7.4

• Bump vulnerable dependencies
• Improve report formatting and some error messages

https://github.com/jfmengels/node-elm-review/compare/v2.7.2...v2.7.3