Skip to content

Conversation

timea-solid
Copy link
Member

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

* better and state of the art configs
* version info
* clean dependencies
* consolidated tests

@timea-solid timea-solid requested a review from Copilot September 17, 2025 14:35
Copy link

@Copilot 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 implements comprehensive improvements to the solid-panes library focused on modernizing the build configuration, cleaning up dependencies, and consolidating test infrastructure. The changes represent a significant upgrade to project tooling and code standards.

  • Modernized build system using TypeScript compiler directly instead of Babel
  • Updated ESLint configuration to use neostandard and improved TypeScript support
  • Consolidated test setup and improved test infrastructure with centralized configuration

Reviewed Changes

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

Show a summary per file
File Description
webpack.config.js Modernized to ES modules and improved output configuration
package.json Updated build scripts, dependencies, and project configuration
eslint.config.mjs Complete rewrite using neostandard with enhanced TypeScript rules
babel.config.js Converted from .babelrc to ES module format
test/helpers/setup.ts New centralized test setup replacing individual test file configurations
tsconfig.test.json New TypeScript configuration specifically for tests
Multiple test files Removed duplicate TextEncoder/TextDecoder setup and fixed import paths
Multiple source files Code style improvements including quote consistency and object shorthand

Tip: Customize your code reviews with copilot-instructions.md. Create the file or 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.

1 participant