Owen Stephens

This feature request came from https://github.com/rubocop/rubocop-factory_bot/pull/52 which has an example reproduction of the problem. ### Problem this feature will solve E.g. with a simple factory: ```ruby factory :post do sequence...

Trailing comment in method body sometimes interferes with smell suppression comment

1

Reek (6.1.4) does not report any smells for the following code: ```ruby class ExampleClass # :reek:LongParameterList private def some_method(first, second, third, fourth, fifth) p 1 end end ``` however, after...

Error when listing columns for model backed by different DB

10

We have a couple of models that are backed by a different DB to the majority of models in our app. A rough example is something like this: ```ruby class...

I noticed that this gem makes a large number of allocations - if we add the `frozen_string_literal` pragma the allocations are vastly reduced. ([See this](https://old.reddit.com/r/ruby/comments/19dt1wl/performance_gains_in_ruby_33_stringdup_vs_string/kjbzjp5/) for a good explanation why)...

Update CI to run against Ruby 3.3, drop Ruby 2.6

4

`google-protobuf` [supports Ruby 3.3 in platform-specific versions of 3.25.3](https://rubygems.org/gems/google-protobuf/versions/3.25.3-x86_64-linux) Some discussion in this thread: https://github.com/protocolbuffers/protobuf/issues/15336#issuecomment-1946383093

**Is your feature request related to a problem? Please describe.** Parlour transitively depends on a version of highline that emits a warning on Ruby 3.3. The [latest release](https://rubygems.org/gems/parlour/versions/8.1.0) of parlour...

Unsubscribe all Rails built-in subscribers

1

Fixes #385 Assumes that a built-in subscriber is nested under the component namespace, so under ActionView or ActionController.

ActionView logs not completely suppressed with Rails 7.1+

1

# Background After this Rails change https://github.com/rails/rails/commit/9c58a54702b038b9acebdb3efa85c26156ff1987 (included in 7.1+) some `ActionView::LogSubscriber`-generated logs have been split out into a separate `ActionView::LogSubscriber::Start` subscriber. Unfortunately, this breaks lograge's unsubscription code invoked here:...

Diff of Range objects is less useful than default RSpec output

2

When an expectation that two Range objects are equal is violated, the super_diff output hides the values within the Ranges: ``` Failure/Error: expect(actual).to eq(expected) Expected # to eq #. Diff:...

Hash include with composable matcher gives misleading diff

3

Expecting a hash to include another when using composable matchers gives a misleading diff for the subset of the hash that _did_ match: ``` 1) includes should include {:a =>...