Skip to content

remove old dev deps#201

Merged
MoOx merged 1 commit into
mainfrom
byemocha
Jun 4, 2026
Merged

remove old dev deps#201
MoOx merged 1 commit into
mainfrom
byemocha

Conversation

@MoOx

@MoOx MoOx commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings June 4, 2026 08:27

Copilot AI left a comment

Copy link
Copy Markdown

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 removes legacy dev-only test dependencies (Mocha/Should) and migrates the test suite to Node’s built-in test runner, reducing external tooling while keeping test coverage in-repo.

Changes:

  • Replaced mocha + should assertions with node:test + node:assert in the existing test file.
  • Removed old devDependencies from package.json and pruned them from package-lock.json.
  • Updated the npm test script to use node --test.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 3 comments.

File Description
test/rework.js Migrates tests from should assertions to node:assert while using node:test primitives.
package.json Drops Mocha/Should dev deps and switches the test script to Node’s test runner.
package-lock.json Removes lockfile entries that were only needed for the deleted devDependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment thread test/rework.js
Comment thread test/rework.js
@MoOx MoOx merged commit 48773e9 into main Jun 4, 2026
9 checks passed
@MoOx MoOx deleted the byemocha branch June 4, 2026 08:47
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