Phil Pirozhkov
Phil Pirozhkov
Do you think there is something actionable here, or can this be closed?
I'm looking at the example docstrings John provided, from the longest, and they fall into several categories: 1. Longest, verbose ``` autocorrects `it "should only fail" do end` to `it...
[RuboCop is red](https://app.circleci.com/pipelines/github/rubocop-hq/rubocop-rspec/743/workflows/9f4f9b76-0879-4ddd-b6f0-41a6e9a6599d/jobs/19251), I hope you'll find it relatively easy to fix it.
Thank you! Looks great 🙌
Thank you for the contribution, your time and effort, @Liberatys! My apologies for delaying the merge.
> Looks like RSpec doesn't run the `after_teardown` hook automatically, so **`TimeHelpers` isn't safe without using blocks** outside of Minitest. Looks like a shortcoming in `rspec-rails`, I believe it should...
Would you be interested in completing this @sambostock ?
It's been a while and I'm lost the context, but I recall there's nothing to fix on `rspec-rails` side.
I side with @sambostock that `rubocop-rails` is a good fit for this cop. Another similar example of a cop would be one that would suggest using `use_transactional_tests` to replace DatabaseCleaner....
> we'll have to only run this cop on public methods Makes sense. > no simple way of detecting if a method is public or not There seems to be...