Skip to content

Conversation

@timea-solid
Copy link
Member

Dropped node 18
improved configs
improved structure

Copilot AI review requested due to automatic review settings October 28, 2025 09:41
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 modernizes the project structure by dropping Node.js 18 support, restructuring the build configuration, and implementing various code improvements including better linting rules, updated dependencies, and code style consistency.

Key Changes

  • Dropped Node 18 support from CI workflow and updated build tooling
  • Restructured file organization with new src/ and dist/ directories
  • Enhanced ESLint configuration with neostandard and TypeScript support
  • Updated import paths to use new directory structure

Reviewed Changes

Copilot reviewed 31 out of 39 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
webpack.config.js Added webpack configuration for building multiple entry points with polyfills
package.json Updated build scripts, dependencies, and main entry point to use dist directory
eslint.config.mjs Replaced old config with comprehensive neostandard-based configuration
babel.config.js Converted to ES module syntax
Makefile Updated build targets to output to dist directory
.github/workflows/ci.yml Removed Node 18 from test matrix
test/unit/setup.ts Moved test setup file from tests to test directory
test/unit/data-reformat-test.test.ts Updated imports and modernized test syntax
src/*.js Updated import paths and applied code style improvements

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

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