8000 Add InstallNpmDevDependencyTaskExecutor & InstallNpmDevDependencyTask · Issue #93 · ibuildingsnl/qa-tools · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Apr 3, 2019. It is now read-only.

Add InstallNpmDevDependencyTaskExecutor & InstallNpmDevDependencyTask #93

Open
rjkip opened this issue Feb 17, 2017 · 1 comment
Open

Add InstallNpmDevDependencyTaskExecutor & InstallNpmDevDependencyTask #93

rjkip opened this issue Feb 17, 2017 · 1 comment
Labels
enhancement
Milestone
3.1.0

Comments

@rjkip
Copy link
Contributor
rjkip commented Feb 17, 2017

Originally submitted by @DRvanR:

Executor should first check whether or not npm is installed, and then also verify whether or not the packages can be installed akin to (but hopefully simpler than) the InstallComposerDevDependencyTaskExecutor. See the NPM --dry-run argument.
@rjkip rjkip added this to the 3.1.0 milestone Feb 17, 2017
@rjkip
Copy link
Contributor Author
rjkip commented Feb 17, 2017

@rpkamp wrote:

WIP PR: #84

This was referenced Feb 17, 2017
Open
Open
rjkip added a commit that referenced this issue Feb 17, 2017
@rjkip
Merge pull request #93 from ibuildingsnl/dont-force-ansi
d5b34e6 
Don't force ANSI colouring onto developers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
3.1.0
Development

No branches or pull requests
1 participant
@rjkip
0