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

Improve packaging of the Electron app #4

Merged
merged 1 commit into from
Jul 15, 2024
Merged

Improve packaging of the Electron app #4

merged 1 commit into from
Jul 15, 2024

Conversation

spa5k
Copy link
Owner

@spa5k spa5k commented Jul 15, 2024

This pull request improves the packaging of the Electron app by making the following changes:

  • Added a homepage URL to the package.json file.

  • Updated the "copy" script in the package.json file to include copying the necessary files from the web directory.

  • Updated the "prepare" script in the package.json file to clean the web directory and copy the necessary files.

  • Updated the "dist" script in the package.json file to run the "prepare" script before building the Electron app.

  • Updated the "linux" script in the package.json file to run the "prepare" script before building the Linux version of the Electron app.

  • Updated the "win" script in the package.json file to run the "prepare" script before building the Windows version of the Electron app.

  • Updated the "mac" script in the package.json file to run the "prepare" script before building the macOS version of the Electron app.

  • Updated the "build" section in the package.json file to include the necessary files and folders for the Electron app.

  • Updated the "main.ts" file to remove unnecessary console.log statements and update the web directory path.

These changes improve the packaging process of the Electron app and ensure that all necessary files are included in the built app.

Thanks to @dnnsjsk

@spa5k spa5k merged commit e1560b5 into main Jul 15, 2024
1 check passed
@spa5k spa5k deleted the next branch July 15, 2024 15:28
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.

1 participant