Skip to content

chore: Replace mocha/expect/proxyquire/nyc with node:test harness - #73

Merged
phated merged 19 commits into
gulpjs:mainfrom
FotoVerite:main
Oct 28, 2026
Merged

phated merged 19 commits into
gulpjs:mainfrom
FotoVerite:main

Conversation

@FotoVerite

Copy link
Copy Markdown
Contributor

closes #72

@FotoVerite
FotoVerite requested a review from phated October 26, 2026 03:07
@FotoVerite FotoVerite self-assigned this Oct 26, 2026
@phated phated added this to v6 Oct 26, 2026
@github-project-automation github-project-automation Bot moved this to Todo in v6 Oct 26, 2026

@phated phated left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Few things here.

Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
@github-project-automation github-project-automation Bot moved this from Todo to In Progress in v6 Oct 26, 2026

@phated phated left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Few things still

Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
Comment thread test/index.js Resolved
@FotoVerite
FotoVerite requested a review from phated October 28, 2026 20:28
Comment thread test/index.js Resolved
@phated phated changed the title chore: Replace mocha/expect/nyc/proxyquire with node:test harness chore: Replace mocha/expect/proxyquire/nyc with node:test harness Oct 28, 2026

@phated phated left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I applied some cleanup that I saw while reviewing so this is good to go. Thanks!

@phated
phated merged commit ed4129d into gulpjs:main Oct 28, 2026
10 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in v6 Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Update test harness

2 participants