fix: re-add runOnce property to CronJob - Githubissues

kelektiv / node-cron

Cron for NodeJS.

MIT License

8.31k stars 617 forks source link

fix: re-add runOnce property to CronJob #751

Closed sheerlox closed 8 months ago

sheerlox commented 8 months ago

Description

add runOnce back to CronJob, this time as a getter to shadowing this.cronTime.realDate value.

Related Issue

https://github.com/kelektiv/node-cron/pull/740#issuecomment-1777662358

Motivation and Context

removing this property was a breaking change.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[ ] If my change introduces a breaking change, I have added a ! after the type/scope in the title (see the Conventional Commits standard).

ncb000gt commented 8 months ago

:tada: This PR is included in version 3.1.5 :tada:

The release is available on:

Your semantic-release bot :package::rocket: