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

Exclude Readme and tests files in external modules #394

Open
anantab opened this issue May 15, 2018 · 5 comments
Open

Exclude Readme and tests files in external modules #394

anantab opened this issue May 15, 2018 · 5 comments
Labels

Comments

@anantab
Copy link

anantab commented May 15, 2018

I do not want external modules to be bundled, but want them to be packaged in artifacts. I can do this by custom option

custom:
  webpack:
    includeModules: true

But this plugin includes all files in the external module. How do I exclude Read me and test files from external modules in the packaged artifact?

@anantab anantab changed the title Exclude Readme and tests file in external modules Exclude Readme and tests files in external modules May 15, 2018
@HyperBrain
Copy link
Member

Hi @anantab , a solution for this is already on the way (see #327 ). I will adapt the PR soon, so that you can sepcify exlusion patterns that will be omitted.

Right now, the complete modules are packaged (the content that is installed with npm install --production or yarn install --production)

@anantab
Copy link
Author

anantab commented May 27, 2018

hi @HyperBrain, any ETA on when this is going to be released.

@HyperBrain
Copy link
Member

Hi @anantab . I was just too busy the last days/week. This is the next thing on the roadmap. I think I can give it a push on the next weekend.

@anantab
Copy link
Author

anantab commented May 28, 2018

Hi @HyperBrain, thank you for the update and thanks for this excellent plugin. Keep up the good work.

@choxnox
Copy link

choxnox commented Dec 1, 2018

Any updates regarding this feature or at least a workaround?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants