-
Notifications
You must be signed in to change notification settings - Fork 1k
Use start
script
#43
Comments
That is, you'd like to just run |
Or some property in the |
You can always specify entry point javascript file explicitly in CLI instead of passing path to |
Thoughts on using the I know it's primarily used in libraries, but it would be a good fit for |
I would still go for the start script entry, also if it's supposed to be a
shell command, because people usually just define an executable there or if
not an ad-hoc executable script can be created. If not, I would add a new
field, but I'm a bit against this idea having currently valid alternatives.
El 2/5/2017 6:28, "Jim Buck" <[email protected]> escribió:
… Thoughts on using the main
<https://docs.npmjs.com/files/package.json#main> field?
I know it's primarily used in libraries, but it would be a good fit for
pkg as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#43 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAgfvsB_MGKMUwiZ6AhEUOJ6pv42_jKBks5r1rD-gaJpZM4NMhwZ>
.
|
Sorry, i can not get why you don't like |
And yeah, you can specify several subfields in |
@igorklopov Ahhh I did not realize that the I'm closing since I don't think it makes sense to duplicate functionality when there is already a mechanism to specify entry point in the |
Please open this again, the problem is not the lack of functionality but
the fact it's not standard compliant, as start script is. Please read npm
package.json documentation and fix that, bin field is not designed to be
used that way and it's a bug.
El 2/5/2017 7:45, "Nathan Rajlich" <[email protected]> escribió:
… @igorklopov <https://github.com/igorklopov> Ahhh I did not realize that
the bin entry was already supported. I see now that it's also documented
in the README <https://github.com/zeit/pkg#usage>.
I'm closing since I don't think it makes sense to duplicate functionality
when there is already a mechanism to specify entry point in the
package.json. Thank you for the suggestion though!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#43 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAgfvkGT919cBfMrlEPWDRJr_PNwiksUks5r1sMVgaJpZM4NMhwZ>
.
|
@piranna The
|
|
Great idea with Implementing that allows us easily pack micro-services to single executable. UPD: As quick fix, it possible to use full path to runner in |
Hello, is it possible to run a global CLI, like "adonis migration: run" from pkg side? |
The canonical default executable is
start
script in thepackage.json
file, that by default isserver.js
file, notbin
entry since you can define several binaries in a single package.The text was updated successfully, but these errors were encountered: