Skip to content

Update README with Rails example#1735

Merged
corincerami merged 4 commits intoresque:masterfrom
karpiu:patch-1
Jul 7, 2021
Merged

Update README with Rails example#1735
corincerami merged 4 commits intoresque:masterfrom
karpiu:patch-1

Conversation

@karpiu
Copy link
Copy Markdown
Contributor

@karpiu karpiu commented Dec 11, 2020

I know the Rails case is in the wiki, but it took me awhile to figure this out, and in the meantime I was stuck. I even posted a question on stackoverflow:

https://stackoverflow.com/questions/65224796/resque-not-failing-an-obviously-buggy-job

And one of the users suggested I request this change, as the Rails is pretty popular case for the Resque gem.

I know the Rails case is in the wiki, but it took me awhile to figure this out, and in the meantime I was stuck. I even posted a question on stackoverflow:

https://stackoverflow.com/questions/65224796/resque-not-failing-an-obviously-buggy-job

And one of the users suggested I request this change, as the Rails is pretty popular case for the Resque gem.
Copy link
Copy Markdown
Contributor

@iloveitaly iloveitaly left a comment

Choose a reason for hiding this comment

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

@karpiu this is great! I think it would fit better as the last section within the "Jobs" header as opposed to right after the intro paragraph:

https://github.com/resque/resque#jobs

Could you move this around and ping me again on the PR?

@karpiu
Copy link
Copy Markdown
Contributor Author

karpiu commented Jul 6, 2021

Addressed comments @iloveitaly

Copy link
Copy Markdown
Contributor

@iloveitaly iloveitaly left a comment

Choose a reason for hiding this comment

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

Great! Thank you.

@karpiu
Copy link
Copy Markdown
Contributor Author

karpiu commented Jul 7, 2021

@chrisccerami done

@corincerami
Copy link
Copy Markdown
Member

Thanks a lot @karpiu

@corincerami corincerami merged commit 29ff6a4 into resque:master Jul 7, 2021
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.

3 participants