Skip to content

Use ESM format by default #71

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

teslor
Copy link

@teslor teslor commented Jan 21, 2025

Update project to use ESM syntax, as it becomes a standard for modern apps, also new versions of many packages require it.

Changes:

  • Update scripts, TS files, tsconfig.json to use ESM.
  • Use node: prefix for node packages as it's good practice; group node modules imports.
  • Update @vitejs/plugin-vue & vite to latest ESM version.
  • Leave CommonJS format for the preload script as it's not treated as ESM by default.

After making these changes, I noticed that there was already a similar PR #6. I guess it's time to make these updates.

@Deluze Deluze self-assigned this Apr 28, 2025
@Deluze Deluze self-requested a review April 28, 2025 09:35
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