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

Use of the electron node #10

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open

Use of the electron node #10

wants to merge 24 commits into from

Conversation

hrenaud
Copy link
Collaborator

@hrenaud hrenaud commented Sep 23, 2024

  • Use Electron (internal) Node ;
  • Use the plugin browser auto-installation ;
  • Update of the collect call in the app.

@hrenaud hrenaud added this to the Lancement milestone Sep 23, 2024
@hrenaud hrenaud added enhancement New feature or request back labels Sep 23, 2024
@hrenaud hrenaud self-assigned this Sep 23, 2024
@hrenaud
Copy link
Collaborator Author

hrenaud commented Sep 23, 2024

L'auto-installation du Browser ne fonctionne pas. → On le lance dans le process d'initialisation.

@hrenaud
Copy link
Collaborator Author

hrenaud commented Sep 29, 2024

Le Node contenu par Electron semble inaccessible, et donc l'usage de spawn n'est pas utilisable.

fork ne semble pas fonctionner.

L'appel direct des méthodes proposées par le plugin parais avoir un problème de path. Il faudrait analyser le code du plugin pour voir si le problème est réel et corrigeable.


L'utilisation du plugin interne permet de gagner du temps au lancement:

  • Pas besoin de check + download du plugin
  • Pas besoin de tester si l'utilisateur peut installer (droit d'écriture)
  • Pas besoin de lire+sauvegarder NPM Dir

@hrenaud
Copy link
Collaborator Author

hrenaud commented Oct 17, 2024

  • fixer le problème de path provenant du plugin lighthouse-plugin-ecoindex

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant