Skip to content

Conversation

@timea-solid
Copy link
Member

@timea-solid timea-solid commented Sep 29, 2025

This is work done to improve different aspects of the library:

  • drop node 18
  • better and state of the art configs
  • clean dependencies, clean scripts
  • ESM style codebase

@timea-solid timea-solid requested a review from Copilot November 3, 2025 11:12
Copy link
Contributor

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 modernizes the codebase by migrating from CommonJS to ES modules, updating configuration files, reorganizing test structure, and standardizing code style.

  • Converts all module syntax from require()/module.exports to import/export
  • Migrates configuration files (Jest, Babel, ESLint) from .js to .mjs format
  • Changes quote style from double quotes to single quotes and removes semicolons
  • Reorganizes tests from src/integration-tests/ to test/ directory
  • Replaces Makefile-based build with direct TTL file imports

Reviewed Changes

Copilot reviewed 65 out of 74 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
webpack.dev.config.js Converted to ES module syntax with import/export
tsconfig.json Updated compiler options and added new configuration files for test/eslint contexts
package.json Updated scripts, dependencies, and removed obsolete ESLint packages
jest.config.mjs New Jest configuration in ES module format
eslint.config.mjs New flat config ESLint setup replacing legacy format
test/**/*.spec.ts Tests moved from src/integration-tests/ and updated to use single quotes
src/**/*.ts All source files reformatted to use single quotes and no semicolons

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

Copy link

Copilot AI commented Nov 3, 2025

@timea-solid I've opened a new pull request, #121, to work on those changes. Once the pull request is ready, I'll request review from you.

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