-
Notifications
You must be signed in to change notification settings - Fork 1
Improvements #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Improvements #94
Conversation
There was a problem hiding this 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 dropping Node 18 support and restructuring the build system to use ES modules and a production-focused webpack configuration.
Key Changes:
- Migrated from CommonJS to ES modules throughout the codebase
- Replaced
vardeclarations withconstandletfor better scoping - Restructured build configuration separating development and production webpack configs
Reviewed Changes
Copilot reviewed 11 out of 14 changed files in this pull request and generated 13 comments.
Show a summary per file
| File | Description |
|---|---|
| webpack.dev.config.js | New development webpack configuration using ES module exports |
| webpack.config.js | Production webpack config with UMD library output and ES module syntax |
| timestamp.sh | Removed build-time version info script |
| src/versionInfo.ts | Removed auto-generated version metadata file |
| src/meetingPane.js | Converted to ES modules, replaced var with const/let, updated string literals |
| package.json | Updated build scripts, main entry point, and dependencies |
| eslint.config.mjs | Reconfigured with neostandard preset and stricter rules |
| dev/index.js | Converted to ES module imports |
| babel.config.js | Created ES module babel config |
| .github/workflows/ci.yml | Removed Node 18 from CI matrix |
| .babelrc | Removed in favor of babel.config.js |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
drop node 18
new structure