Skip to content

Commit

Permalink
dev-cli: allow to set the interpreter (#3100)
Browse files Browse the repository at this point in the history
  • Loading branch information
vmarchaud committed Aug 21, 2017
1 parent cc0f0aa commit 74dddea
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions lib/binaries/DevCLI.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ commander.version(pkg.version)
.option('--post-exec [cmd]', 'execute extra command after change detected')
.option('--silent-exec', 'do not output result of post command', false)
.option('--test-mode', 'debug mode for test suit')
.option('--interpreter <interpreter>', 'the interpreter pm2 should use for executing app (bash, python...)')
.option('--env [name]', 'select env_[name] env variables in process config file')
.option('--auto-exit', 'exit if all processes are errored/stopped or 0 apps launched')
.usage('pm2-dev app.js');
Expand Down

4 comments on commit 74dddea

@uyu423
Copy link

@uyu423 uyu423 commented on 74dddea Aug 21, 2017

Choose a reason for hiding this comment

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

Is it only necessary to add the string associated with the option? I thought there would be a more complicated process. (#3100)

@vmarchaud
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In this case no because its already implemented inside the pm2 code and pm2-dev is automatically giving the key/value to PM2 code :)

@uyu423
Copy link

@uyu423 uyu423 commented on 74dddea Aug 21, 2017

Choose a reason for hiding this comment

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

@vmarchaud The structure is very amazing. If I analyzed the code a little more carefully, I think I might have committed. I really appreciate your explanation. It would be nice if sometime my code could be included in pm2. I should try harder.

@vmarchaud
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@uyu423 Just pick a issue with Open for PR tag and try to fix it, if you want i can explain a bit to guide you

Please sign in to comment.