Skip to content

Fix dev dependency exclusion bug when too many node modules are used #3874

@pmuens

Description

@pmuens

This is a Bug Report

Description

Right now we do a glob exclude of the whole node_modules directory and include only prod dependencies by using npm ls to identify them and a glob to re-include them.

However there's a bug with the maximum number of characters the buffer can hold.

Furthermore we should switch the behavior. We should list the dev dependencies and exclude them (while still including the whole node_modules directory) to ensure that the whole node_modules directory is still in included. Even if the command fails.


Quick fix / other addition

Add a config parameter which allows people to opt out of this feature.


Related #3827

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions