Skip to content

[fix] cron should gracefully reload process (closes #5636)#5637

Open
titanism wants to merge 1 commit intoUnitech:masterfrom
titanism:patch-1
Open

[fix] cron should gracefully reload process (closes #5636)#5637
titanism wants to merge 1 commit intoUnitech:masterfrom
titanism:patch-1

Conversation

@titanism
Copy link

Per #5636. This should be at minimum a minor semver bump, but recommended major semver bump.

Per <Unitech#5636>.  This should be at minimum a minor semver bump, but recommended major semver bump.
@CLAassistant
Copy link

CLAassistant commented Jul 10, 2023

CLA assistant check
All committers have signed the CLA.

Copy link

@hinogi hinogi left a comment

Choose a reason for hiding this comment

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

LGTM

@rtizzy
Copy link

rtizzy commented Oct 3, 2023

Great work! Just happened to run into this myself while debugging some issues.

Is this going to be brought into mainline anytime soon?

@titanism
Copy link
Author

titanism commented Oct 3, 2023

@rtizzy don't forget to set pmx: false to avoid the PM2 memory leak too FYI, these both are unresolved still, not sure if @Unitech is still maintaining PM2 actively or wants help - just let us know if so, we are "titanism" on npm/GitHub and maintain a ton of projects including @forwardemail @spamscanner @ladjs and lots of @koajs stuff.

#5145 (comment)

@titanism
Copy link
Author

titanism commented Dec 7, 2023

ping @Unitech

1 similar comment
@titanism
Copy link
Author

ping @Unitech

@titanism
Copy link
Author

titanism commented Jun 6, 2024

Ping @Unitech

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants