Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ace-http command doesn't install #51

Closed
rdeltour opened this issue Oct 4, 2017 · 1 comment
Closed

ace-http command doesn't install #51

rdeltour opened this issue Oct 4, 2017 · 1 comment
Assignees
@marisademeglio
Labels
bug
Milestone
Beta 1

Comments

@rdeltour
Copy link
Member

rdeltour commented Oct 4, 2017

Installing globally [email protected] installs the ace command but not ace-http.

To be investigated. Some leads:

  • in any case, add a node shebang at the top of the CLI implementation
  • make the file executable? (I don't think this is needed, to be verified)
  • make sure multiple binaries are supported (AFAIK it is)
@rdeltour rdeltour added the bug label Oct 4, 2017
@rdeltour rdeltour added this to the Beta 1 milestone Oct 4, 2017
marisademeglio added a commit that referenced this issue Oct 5, 2017
@marisademeglio
fix(http): make ace-http command executable
564f695 
According to https://docs.npmjs.com/files/package.json#bin, this should
be sufficient.

Still needs testing.

Addresses #51
@marisademeglio
Copy link
Member

Tested release 0.3.1. This command works now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@marisademeglio marisademeglio

Labels
bug
Projects
None yet
Milestone
Beta 1
Development

No branches or pull requests
2 participants
@rdeltour @marisademeglio