jfmengels / node-elm-review

CLI for elm-review
https://package.elm-lang.org/packages/jfmengels/elm-review/latest/
BSD 3-Clause "New" or "Revised" License
48 stars 25 forks source link

chore(deps): bump jfmengels/elm-review-unused from 1.1.8 to 1.2.0 in /review #205

Closed dependabot[bot] closed 4 weeks ago

dependabot[bot] commented 4 months ago

Bumps jfmengels/elm-review-unused from 1.1.8 to 1.2.0.

Changelog

Sourced from jfmengels/elm-review-unused's changelog.

[1.2.0] - 2023-07-28

Add configuration setting for [NoUnused.Exports] for reporting exports that is only used in non-production code. This includes multiple new functions and types in that module centered around reportUnusedProductionExports.

[1.1.30] - 2023-06-16

Fix false positive in [NoUnused.CustomTypeConstructors] related to non-ASCII constructor names (#79). Thanks @​lydell and @​marc136.

[1.1.29] - 2022-12-07

Fix false positive in [NoUnused.CustomTypeConstructors].

[1.1.28] - 2022-11-08

Add better support for jfmengels/elm-review v2.10.0.

[1.1.27] - 2022-09-19

This release contains HUGE performance updates. Rough benchmarks show some of the slow rules ([NoUnused.Exports], [NoUnused.CustomTypeConstructors] and [NoUnused.CustomTypeConstructorArgs]) are now 20-40 times faster compared to the previous release.

[1.1.26] - 2022-09-11

  • Fixed an issue in [NoUnused.Dependencies] that led the elm.json to become corrupt

[1.1.25] - 2022-09-10

  • [NoUnused.Patterns] now reports multiple aliases (((A a) as y) as z)
  • [NoUnused.Patterns] now reports aliases on a variable pattern (name as otherName)
  • [NoUnused.Patterns] now reports aliases to wildcard (_ as thing) in parameters, and [NoUnused.Parameters] now doesn't
  • Improved the fixes for [NoUnused.Patterns] to not include unnecessary patterns
  • Improved the fixes for [NoUnused.Patterns] to have formatting look more like elm-format

[1.1.24] - 2022-09-02

  • [NoUnused.Variables] now reports imports that get shadowed by other imports ([252475888b79a88f107571c1002d0ed650622ddb])

[1.1.23] - 2022-08-24

This version merges the [NoUnused.Modules] into the [NoUnused.Exports] rule.

A common issue when running elm-review --fix (or --fix-all) was when you had the two rules enabled and encountered an unused module.

While in fix mode, NoUnused.Exports would remove every export one at a time, which would likely be followed by [NoUnused.Variables] removing the previously exported element. This would go on until the module is as empty as it can be. At this point, you would finally be able to see NoUnused.Modules's error indicating that the module is unused.

Whether you want to remove the module or use it somewhere in response to this message, this is a lot of unnecessary work for you and/or the tool, making --fix-all painfully long.

By having the NoUnused.Exports do the work of both rules, and not reporting any unused exports when the entire module

... (truncated)

Commits


Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Note Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

lishaduck commented 4 months ago

Should be good to merge.

jfmengels commented 4 weeks ago

@dependabot rebase

dependabot[bot] commented 4 weeks ago

Looks like jfmengels/elm-review-unused is up-to-date now, so this is no longer needed.