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

Add fusionrc option to override main entry point #736

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pheew
Copy link

@pheew pheew commented Apr 9, 2020

This will allow implementers to use a custom entry point. This opens up the ability to use FusionJS with Typescript or any other language that has babel support. I wanted to get some early feedback before I

  • Add config validation
  • Add documentation
  • Write tests

^^ Am I forgetting anything else?

Fixes: #718

@CLAassistant
Copy link

CLAassistant commented Apr 9, 2020

CLA assistant check
All committers have signed the CLA.

@@ -132,9 +132,14 @@ function getWebpackConfig(opts /*: WebpackConfigOpts */) {
legacyPkgConfig = {},
worker,
} = opts;
const main = 'src/main.js';
const main = fusionConfig.main || 'src/main.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we instead just check for the existence of src/main.js or src/main.ts and update our webpack accordingly?

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.

Allow a dynamic entry point and fall back to main.js if no value is defined.
3 participants