Skip to content

Conversation

afonsojramos
Copy link
Member

This pull request migrates the Electron app's build system from Webpack to electron-vite, resulting in a significant simplification of the configuration files and build scripts. It removes all Webpack-related configuration and dependencies, introduces new vite-based configuration, and updates file paths and environment variable handling throughout the codebase. Additionally, the CSS and asset handling are modernized to work with Vite and Tailwind v4 (no need for a config file), and logger usage is improved for better compatibility between main and renderer processes.

At the current state however, Primer's variables are being loaded, but somehow not used yet.

@github-actions github-actions bot added dependency Dependency updates enhancement New feature or enhancement to existing functionality labels Sep 15, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
50.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@import "@primer/css/dist/primer.css";

/** Tailwind CSS Configuration */
@config '../../tailwind.config.ts';
Copy link
Collaborator

@bmulholland bmulholland Sep 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It it possible to make the tailwind change separately, to reduce complexity and risk for both vite and tailwind changes?

@bmulholland
Copy link
Collaborator

I have a working project with electron-vite in use. I'm not super experienced with it, but may be able to help if you run into something tricky.

@afonsojramos
Copy link
Member Author

I have a working project with electron-vite in use. I'm not super experienced with it, but may be able to help if you run into something tricky.

The goal was to simply simplify the project as webpack does present its challenges, so hopefully less tricky things will appear 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency Dependency updates enhancement New feature or enhancement to existing functionality
Development

Successfully merging this pull request may close these issues.

2 participants