Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Creates bin package and folder - Closes #35 #36

Merged
merged 5 commits into from Jul 10, 2017
Merged

Conversation

toschdev
Copy link
Contributor

Closes #35

@toschdev toschdev self-assigned this Jun 22, 2017
@karmacoma karmacoma removed the *easy label Jun 29, 2017
@karmacoma karmacoma added this to Pending Review in Version 0.1.0 Jun 29, 2017
@toschdev toschdev added the *easy label Jul 6, 2017
@willclarktech
Copy link
Contributor

Do we need to add a postinstall script running npm link?

@willclarktech willclarktech self-requested a review July 10, 2017 09:35
package.json Outdated
"scripts": {
"start": "node index.js",
"test": "NODE_ENV=test nyc mocha",
"postinstall": "npm link"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comma

bin/lisky Outdated
@@ -0,0 +1,3 @@
#!/usr/bin/env node

module.exports = require('../index.js');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer without the .js extension?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants