Skip to content

Conversation

@hailiang-wang
Copy link
Contributor

In package.json, add cleanup script.

    "cleanup": "DEBUG=* ./superscript/lib/bin/cleanup.js --mongo PhillyLogicDB",

Expected Behavior

All data in mongodb is dropped.

Current Behavior

Get an error.

$ npm run cleanup

> philly-logic@1.0.0 cleanup /root/git/chatbot-mvp/philly-logic
> DEBUG=* ./superscript/lib/bin/cleanup.js --mongo PhillyLogicDB

sh: 1: ./superscript/lib/bin/cleanup.js: Permission denied

npm ERR! Linux 3.13.0-86-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "cleanup"
npm ERR! node v6.9.1
npm ERR! npm  v3.10.8
npm ERR! code ELIFECYCLE
npm ERR! philly-logic@1.0.0 cleanup: `DEBUG=* ./superscript/lib/bin/cleanup.js --mongo PhillyLogicDB`
npm ERR! Exit status 126
npm ERR!
npm ERR! Failed at the philly-logic@1.0.0 cleanup script 'DEBUG=* ./superscript/lib/bin/cleanup.js --mongo PhillyLogicDB'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the philly-logic package,

Possible Solution

Add exec permissions.

Steps to Reproduce (for bugs)

Context

Your Environment

  • Version used:
  • Environment name and version (e.g. PHP 5.4 on nginx 1.9.1):
  • Server type and version:
  • Operating System and version:
  • Link to your project:

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 65.448% when pulling 3d3be78 on Samurais:v1.0.0 into ffef2e2 on superscriptjs:v1.0.0.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 65.448% when pulling 3d3be78 on Samurais:v1.0.0 into ffef2e2 on superscriptjs:v1.0.0.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 65.448% when pulling 3d3be78 on Samurais:v1.0.0 into ffef2e2 on superscriptjs:v1.0.0.

@bensalilijames bensalilijames merged commit 2358c40 into superscriptjs:v1.0.0 Nov 16, 2016
@bensalilijames
Copy link
Collaborator

Good catch - thanks!

duffrind pushed a commit to duffrind/superscript that referenced this pull request Aug 29, 2017
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.

3 participants