Skip to content

Remove build files from repository #92

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: dev
Choose a base branch
from

Conversation

Borewit
Copy link

@Borewit Borewit commented Jul 17, 2025

This PR removes the build artifacts from version control. These files are typically generated during the build process and are not necessary to track in Git.

Benefits:

  • Reduces repository size and clutter.
  • Avoids outdated or conflicting build artifacts in pull requests.
  • Pull requests are cleaner and more focused on actual source changes.
  • Reduces chances of merge conflicts
  • Follows common best practices for modern JavaScript libraries.

The build artifacts can still be produced using npm run build

@Borewit Borewit force-pushed the remove-build-files branch from 480a700 to 4e7165c Compare July 17, 2025 12:37
@hvianna
Copy link
Owner

hvianna commented Jul 17, 2025

Originally the app had to be hosted in a web server (the support for the FileSystemAccess API was added just a year ago), so the bundled files were included for users that didn't have experience with node/npm.

I agree with the benefits of removing them now, but I'd need to check with our friend @JesusGod-Pope666-Info if he's ready to install Node and try bundling it himself :)

Ignore custom user configuration file
@Borewit Borewit force-pushed the remove-build-files branch from 4e7165c to 55026d6 Compare July 19, 2025 15:02
@Borewit
Copy link
Author

Borewit commented Jul 19, 2025

Why not build the distribution in a pipeline (Git workflow)? That way you can also exclude files which do not belong in there.

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