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

Swap Webpack with Parcel #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bigtimebuddy
Copy link

This template is super useful for getting started. I thought it might be dramatically simplified by using Parcel. This would avoid the boilerplate code and multiple dependencies for Webpack which is confusing for the uninitiated. Parcel has serving, minifying, and watching out-of-the-box with no other config.

Lately, I've been favoring Parcel for examples/demos because of how it's so straightforward and readable.

@miltoncandelero
Copy link
Owner

miltoncandelero commented Oct 3, 2022

this template started with parcel v1 because it was "configless"
in parcel v2 it started requiring a configuration file and at that point why not go with the industry standard webpack? 🤷

I will check this out when I get the time, it has to do two things:

  • Keep assets separated from code: no direct imports or base64s, leverage the pixi loader (or assets, in the future)
  • Make standalone builds: No need to go asset hunting, the dist folder must have the entire game ready to upload.

@bigtimebuddy
Copy link
Author

bigtimebuddy commented Oct 3, 2022

I think parcel v2 is very solid. No config required, super fast, just a source field in package.json. I like managing assets the parcel way instead of pathing and copying assets, which is easy to mess up at runtime, instead of at build-time. Also, since parcel adds a hash to the file is a little better for managing browser caching.

Up to you. My feelings aren't hurt either way.

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