Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace broken croniter #273

Merged
merged 2 commits into from
Oct 28, 2021
Merged

Replace broken croniter #273

merged 2 commits into from
Oct 28, 2021

Conversation

urm8
Copy link
Contributor

@urm8 urm8 commented Oct 27, 2021

regarding this issues: #250, #260, for some reason,
for cronstring: 5 * * * * *
for local time: x:5:*, x in [0-24]
croniter yields x.5.01

@selwin do you actively maintain this repo ?

@selwin selwin merged commit b1e3727 into rq:master Oct 28, 2021
@selwin
Copy link
Contributor

selwin commented Oct 28, 2021

@urm8 thanks for the PR. Unfortunately not much work is going into this repo as RQ already has a built in scheduler (though not as fully featured as this repo because there are somethings that don't fit within the paradigm).

I'm happy to make releases every now and then to fix bugs though.

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.

2 participants