Skip to content
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

Minify main.js to improve plugin load time #163

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Kageetai
Copy link

@Kageetai Kageetai commented Nov 25, 2024

Fixes #161

Add minification to the build process for main.js.

  • rollup.config.js

    • Import esbuild from rollup-plugin-esbuild and terser from rollup-plugin-terser.
    • Add esbuild and terser to the plugins array for minification.
  • package.json

    • Add rollup-plugin-esbuild as a devDependency.

For more details, open the Copilot Workspace session.

Fixes argenos#161

Add minification to the build process for `main.js`.

* **rollup.config.js**
  - Import `esbuild` from `rollup-plugin-esbuild` and `terser` from `rollup-plugin-terser`.
  - Add `esbuild` and `terser` to the `plugins` array for minification.

* **package.json**
  - Add `rollup-plugin-esbuild` as a devDependency.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/argenos/nldates-obsidian/issues/161?shareId=XXXX-XXXX-XXXX-XXXX).
* Add `rollup-plugin-terser` to `devDependencies`
* Remove `rollup-plugin-esbuild` from `devDependencies`
* Add `rollup-plugin-esbuild` to `dependencies`
* Add `rollup-plugin-terser` to `dependencies`
* Add various new dependencies and sub-dependencies in `package-lock.json`
* Update existing dependencies and sub-dependencies in `package-lock.json`
…ckage-lock.json`

* Remove `rollup-plugin-esbuild` entry from `dependencies` in `package.json`
* Remove `rollup-plugin-esbuild` and its related dependencies from `package-lock.json`
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.

Minify main.js
1 participant