Update dependency elm-review to v2.5.3

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
elm-review	2.5.2 -> 2.5.3

---

Release Notes

jfmengels/node-elm-review

v2.5.3

Compare Source

• Speed up calling elm-format by not using npx (#​49) 5669aef

There is a pretty big delay in calling elm-format when it's done through npx like we were doing. @​lydell made it so we don't use npx, which reduces the amount of time it takes to apply fixes. You'll notice it most when using --fix-all when a lot of files get changed.

The time spent calling elm-format went (when fixing 4 files) from about 1500ms to 300-400ms, and sometimes even around 150ms, depending on where elm-format is in your system.

This change is a bit scary, and it might break in some edge cases, but we don't expect anyone to be in those cases. If you do, please open an issue and temporarily go back to 2.5.2.

• Fix command injection issue (Do what we can to avoid command injection jfmengels/node-elm-review#46) 0751825

@​lydell also noticed a security issue when applying fixes (or rather, when spawning elm-format), in very special and unlikely circumstances.

• Improve error message when entering an invalid rule name (#​48) 9ce7dd2

I also improved the error message when using non-ASCII characters in the rule name for new-package or new-rule. Thanks to @​lydell for noticing!

Thanks @​lydell for all the awesome work!

---

Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box.

---

This PR has been generated by WhiteSource Renovate. View repository job log here.