pm2 feat: stop-exit-codes

Q A

Bug fix? yes (adding a missing feature mentioned in the docs)

New feature? yes

BC breaks? no

Deprecations? no

Tests pass? yes

Fixed tickets #5208

License MIT

Doc PR https://github.com/pm2-hive/pm2-hive.github.io/pull/218 (already added)

Q	A
Bug fix?	yes (adding a missing feature mentioned in the docs)
New feature?	yes
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	#5208
License	MIT
Doc PR	https://github.com/pm2-hive/pm2-hive.github.io/pull/218 (already added)

This feature stop-exit-codes was first introduced by @jack1142 in the PR #5048 along with the documentation change PR https://github.com/pm2-hive/pm2-hive.github.io/pull/218. However for some reason, only the documentation PR was merged without the actual code and now the live documentation website lists a feature that doesn't actually exist. This of course resulted in this issue #5208 being created.

This PR is continued from jack's PR and adds a few tests. Someone else already made an attempt in #5241 but seems like it was closed without any reason.

Mar 18 '22 23:03 Lazyuki

All committers have signed the CLA.

Mar 18 '22 23:03 CLAassistant

Would love to see this merged please, will help us out a lot 👍

Apr 19 '22 14:04 jrtg

Yes please. EDIT: I manually changed these files to match the PR, but it still doesn't seem to be working for me. :(

Apr 30 '22 02:04 dperolio

Can this be merged please ?

Jun 03 '22 02:06 internpoon

This feature should be merged, could somebody provide feedback so that it can move forward?

Nov 04 '22 09:11 negreanucalin

@Unitech any guidance here?

Nov 06 '22 01:11 TotalLag

This is already mentioned in the docs, but the feature itself does not seem to exist yet. I think this should be removed from the docs until this gets merged.

Dec 04 '22 20:12 bramvbilsen

In docs, passes tests but not merged?

Considering the fast-pace of Node ecosystem in general, I think this project can do with a bias towards shipping and letting the community deal with potential regressions (that too solved with a bias towards shipping).

Please merge this so the featureset is in-line with current documentation, please. It has been 7 months already.

Thanks

Dec 29 '22 16:12 smashah

feature is mentioned in docs since 2021, this pull request / patch / bugfix is nearly a year old...

really like to see this feature (or at least removed from docs...)

I am quite confused about this. considered switchich from supervisor to pm2 but things like leave me wondering how solid this project is

installing the fork this works fine for my usecase for dispatching forked scripts that should not be restarted when run once successful (app booting database migration, initial caching, etc.)