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

Add types to `extra-files.js` #181

Closed lishaduck closed 5 months ago

lishaduck commented 5 months ago

Depends on #175. Here's the diff as GH doesn't play well with stacks from forks: https://github.com/lishaduck/node-elm-review/compare/bump-turbo...extra-files-types

socket-security[bot] commented 5 months 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-config-turbo@2.0.5 Transitive: environment, eval, filesystem, shell +2 477 kB turbobot
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-config-turbo@2.0.4, npm/eslint-plugin-node@11.1.0

View full report↗︎

jfmengels commented 5 months ago

I misdid the order of my Git operations, so it looks closed but this was merged :smile:

Thanks for all the PRs @lishaduck!

jfmengels commented 5 months ago

Do you know if the remaining open PRs are still relevant or if I should close them?

lishaduck commented 5 months ago

Do you know if the remaining open PRs are still relevant or if I should close them?

As in #132? I don't think it's still relevant. The Dependabot ones are probably fine if they're rebased. #147 could be closed, but until I've landed a replacement, I figure we might as well keep it open.