Skip to content

Conversation

@josebolos
Copy link
Contributor

@josebolos josebolos commented Jul 3, 2025

  • Update test matrix from Node 16 to a supported version of Node.js (v20).
  • Reorder list of devDependencies and update all devDependencies to latest version that doesn't require ESM.

@josebolos josebolos requested a review from Copilot July 3, 2025 14:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates testing and mocking libraries to their latest versions in devDependencies.

  • Bumps mocha from ~3.0.2 to ~11.7.1
  • Bumps chai from ~3.0.0 to ~5.2.0
  • Bumps mockery from ~1.7.0 to ~2.1.0
  • Bumps cheerio from ~0.22.0 to ~1.1.0
Comments suppressed due to low confidence (1)

package.json:49

  • After upgrading major versions of testing frameworks, ensure the existing test suite passes and add compatibility tests for any breaking changes in Mocha or Chai.
    "mocha": "~11.7.1",

"mocha": "~3.0.2",
"chai": "~3.0.0",
"mockery": "~1.7.0",
"mocha": "~11.7.1",
Copy link

Copilot AI Jul 3, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider using caret ranges (e.g., ^11.7.1) instead of tilde to receive minor and patch updates automatically.

Suggested change
"mocha": "~11.7.1",
"mocha": "^11.7.1",

Copilot uses AI. Check for mistakes.
package.json Outdated
Comment on lines 48 to 53
"jshint": "~2.13.6",
"mocha": "~3.0.2",
"chai": "~3.0.0",
"mockery": "~1.7.0",
"mocha": "~11.7.1",
"chai": "~5.2.0",
"mockery": "~2.1.0",
"spooks": "~2.0.0",
"cheerio": "~0.22.0"
"cheerio": "~1.1.0"
Copy link

Copilot AI Jul 3, 2025

Choose a reason for hiding this comment

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

[nitpick] Sort devDependencies alphabetically to improve readability and maintainability.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants