Skip to content

Conversation

KurtPreston
Copy link
Contributor

Purpose / Goal

Yesterday, I submitted a PR updating the package.json to include some files missing to get the CLI working. Running the updated code today, I realized there were some other issues, some of which prevented install. This addresses them.

Apologies for not catching these issues yesterday.

There are some other enhancements I may submit to the CLI tools later, but I first wanted to make sure to fix the issues I created.

Type

[x]Bug Fix
[ ]Refactoring / Technology upgrade
[ ]New Feature

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.487% when pulling 8a14437 on KurtPreston:fixPackageFiles into daa711c on NaturalIntelligence:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.487% when pulling 8a14437 on KurtPreston:fixPackageFiles into daa711c on NaturalIntelligence:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 97.487% when pulling 8a14437 on KurtPreston:fixPackageFiles into daa711c on NaturalIntelligence:master.

@SameerSiddiqui
Copy link

When this is going in ?

@SameerSiddiqui
Copy link

npm install --no-progress --no-optional --ignore-scripts --no-shrinkwrap

Doesn't work...

@KurtPreston
Copy link
Contributor Author

@SameerSiddiqui until this gets merged, I'd suggest using the previous release 3.12.0

@amitguptagwl amitguptagwl merged commit 08ffc48 into NaturalIntelligence:master Sep 11, 2018
@amitguptagwl
Copy link
Member

@KurtPreston Thanks for the immediate action.

published

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.

4 participants