Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@
"coveralls": "^3.0.2",
"function-bind": "^1.0.2",
"karma": "^3.0.0",
"karma-browserify": "^5.0.1",
"karma-browserify": "^8.0.0",
Comment on lines 64 to +65
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This upgrade introduces a version compatibility issue. karma-browserify@8.0.0 requires karma@6.x or higher, but the project currently uses karma@3.0.0. To resolve this, either:

  1. Keep karma-browserify@5.x, or
  2. Upgrade karma to version 6.x or higher

The current combination of versions will cause runtime errors.

Spotted by Graphite Reviewer

Is this helpful? React πŸ‘ or πŸ‘Ž to let us know.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The peer dependencies of karma-browserify@8.0.0 are "karma": ">=4.3.0".

The graphite app did not explain why it thought that karma@6 is required. It could be true, but we don't know why.

"karma-cli": "^2.0.0",
"karma-coverage": "^2.0.3",
"karma-phantomjs-launcher": "^1.0.0",
Expand Down
Loading