search

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
47 stars 25 forks source link

More Never #179

Closed lishaduck closed 5 days ago

lishaduck commented 1 week ago

Explicitly state "NoReturns"

Depends on #174. Here's the diff as GH doesn't play well with stacks from forks: https://github.com/lishaduck/node-elm-review/compare/help-refactor...never

socket-security[bot] commented 1 week ago

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@eslint-community/eslint-plugin-eslint-comments@4.3.0 None +2 97.9 kB eslint-community-bot
npm/@typescript-eslint/eslint-plugin@6.21.0 Transitive: environment, filesystem +40 7.24 MB jameshenry
npm/@typescript-eslint/parser@6.21.0 Transitive: environment, filesystem +32 2.55 MB jameshenry
npm/eslint-plugin-n@16.6.2 filesystem Transitive: environment, unsafe +20 2.17 MB weiran.zsd
npm/eslint-plugin-promise@6.2.0 None 0 69.6 kB eslint-community-bot
npm/eslint-plugin-security@2.1.1 None +2 468 kB eslint-community-bot

🚮 Removed packages: npm/@typescript-eslint/eslint-plugin@5.62.0, npm/@typescript-eslint/parser@5.62.0, npm/eslint-plugin-node@11.1.0

View full report↗︎

lishaduck commented 1 week ago

I bumped into typescript-eslint/typescript-eslint#3455, but TS had a similar enough rule I disabled the eslint rule and turned on the tsc rule.

jfmengels commented 5 days ago

Thank you for your work on this @lishaduck!