Skip to content

Commit 2cf263a

Browse files
justin808claude
andcommitted
Add jest-junit to knip ignoreDependencies
jest-junit is used as a Jest reporter in CI but not directly imported, so knip flags it as unused. Add it to ignoreDependencies in both the root workspace and react-on-rails-pro-node-renderer workspace. Also remove unused @babel/runtime and mini-css-extract-plugin entries from spec/dummy ignoreDependencies that knip correctly identified as no longer needed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent 10bc35b commit 2cf263a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

knip.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ const config: KnipConfig = {
3333
'@eslint/js',
3434
// used by Jest
3535
'jsdom',
36+
'jest-junit',
3637
// This is an optional peer dependency because users without RSC don't need it
3738
// but Knip doesn't like such dependencies to be referenced directly in code
3839
'react-on-rails-rsc',
@@ -75,6 +76,8 @@ const config: KnipConfig = {
7576
// Optional dependencies used in integrations
7677
'@honeybadger-io/js',
7778
'@sentry/*',
79+
// Jest reporter used in CI
80+
'jest-junit',
7881
],
7982
},
8083

@@ -170,9 +173,6 @@ const config: KnipConfig = {
170173
'url-loader',
171174
// Transitive dependency of shakapacker but listed as direct dependency
172175
'webpack-merge',
173-
// Dependencies not detected in production mode (runtime injected or dynamic imports)
174-
'@babel/runtime',
175-
'mini-css-extract-plugin',
176176
],
177177
},
178178
},

0 commit comments

Comments
 (0)