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

Update clockwork #654

Merged
merged 2 commits into from
Nov 25, 2016
Merged

Update clockwork #654

merged 2 commits into from
Nov 25, 2016

Conversation

knoxknox
Copy link
Contributor

Update link and description for gem 'clockwork'.
Looks like this gem is not maintained by original authors.

@markets
Copy link
Owner

markets commented Nov 19, 2016

hi @knoxknox, but the original source points out:

This fork is no longer maintained, maybe try this one: https://github.com/Rykian/clockwork

But, in this fork I also found this: Rykian/clockwork#1.

So I'm not sure if this is really an option right now, if we take into account "competitors" like whenever, rufus-scheduler... Keep a link to a 404 isn't really good.

@knoxknox
Copy link
Contributor Author

Hi @markets, yes I saw this, but this is one of the authors of this gem.

What is your suggestion ?
Also I'm not sure about rubygems. Looks like version in rubygems also is not maintained.

@markets
Copy link
Owner

markets commented Nov 21, 2016

Yes, there is actually some uncertainty... I also found this thread on Reddit: https://m.reddit.com/r/rails/comments/5a68on/what_happened_to_the_clockwork_gem

So, probably right now best movement is to remove it from the list (we can add it back in case the project "revives" again). Btw, zhong seems a good lib for a distributed solution: https://github.com/nickelser/zhong (#657)

@knoxknox
Copy link
Contributor Author

@markets done, gem was removed from list

@markets
Copy link
Owner

markets commented Nov 25, 2016

Thanks @knoxknox, this is a proper move IMHO!

There is also a discussion here, in the original repo: adamwiggins/clockwork#23

@markets markets merged commit abe5d98 into markets:master Nov 25, 2016
@markets markets removed the addition label Jan 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants