Skip to content

simplify deps & dev deps + node 22#193

Merged
MoOx merged 2 commits into
mainfrom
simplify-dev-deps
Jun 15, 2026
Merged

simplify deps & dev deps + node 22#193
MoOx merged 2 commits into
mainfrom
simplify-dev-deps

Conversation

@MoOx

@MoOx MoOx commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

Breaking changes, but node <= 20 are at EOL.

Copilot AI review requested due to automatic review settings June 15, 2026 06:34

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 aims to simplify the development toolchain by removing Mocha/Chai/ESLint, migrating tests to Node’s built-in node:test runner, and switching linting to oxlint.

Changes:

  • Replace Mocha/Chai-based test setup with node:test + node:assert (with a small compatibility shim).
  • Replace ESLint with oxlint, and remove ESLint config files.
  • Minor refactors/cleanups in tests and URL/path handling (regex simplifications, startsWith, endsWith, slash normalization).

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/type/inline.js Updates assertions and simplifies a custom filter function.
test/type/copy.js Cleans up regex/string escaping for URL path expectations.
test/setup.js Migrates test harness from Chai/Mocha globals to node:test + node:assert.
test/.eslintrc.js Removes test-specific ESLint config.
src/lib/paths.js Small simplifications in path normalization and URL checks.
package.json Drops mocha/chai/eslint, adds oxlint, updates test/lint scripts, bumps postcss-import.
package-lock.json Updates lockfile to reflect new dev dependency set.
.eslintrc.js Removes root ESLint configuration.

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

Comment thread package.json
@MoOx MoOx changed the title simplify dev deps simplify deps & dev deps + node 22 Jun 15, 2026
@MoOx MoOx merged commit 89f6763 into main Jun 15, 2026
6 checks passed
@MoOx MoOx deleted the simplify-dev-deps branch June 15, 2026 06:54
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