active-job-style-guide icon indicating copy to clipboard operation
active-job-style-guide copied to clipboard

Rubocop extension

Open tejasbubane opened this issue 2 years ago • 1 comments

Thanks for this styleguide. I learnt a few good practices and sidekiq's features.

Does rubocop extension corresponding this style-guide already exist somewhere? I could not find any mentioned in this guide, nor in the rubocop docs.

I am planning to create such extension. Not all rules can be checked by rubocop, but some sure can and that would be helpful. I thought of checking with maintainers here before creating the new extension to avoid duplicate work.

tejasbubane avatar Nov 04 '21 17:11 tejasbubane

There were a few internal cops as far as I remember.

  1. Models as arguments. As I recall, it was prone to false positives. E.g. stripe_id is not an id of a Stripe model.
  2. Queue assignments. Quite useful.

@bbatsov Are there any plans to make those cops available to the public?

I am planning to create such extension

Fantastic!

There's also this bag of notes if you're up to extend the guide itself.

On a side note, I don't have enough capacity to continue building and contributing to this guide myself, but I'll be happy to maintain it under @rubocop organization.

pirj avatar Nov 04 '21 18:11 pirj

An extension exists it turns out https://github.com/dvandersluis/rubocop-sidekiq/ thanks to @dvandersluis I'll close this as completed.

pirj avatar May 10 '23 17:05 pirj