Andy Waite
Andy Waite
cc @paracycle #### Input [→ View on sorbet.run](https://sorbet.run/?arg=--enable-experimental-requires-ancestor#%23%20typed%3A%20true%0A%0Arequire%20%22sorbet-runtime%22%0A%0Aclass%20Foo%0A%20%20extend%20T%3A%3ASig%0A%20%20extend%20T%3A%3AHelpers%0A%0A%20%20abstract!%0A%0A%20%20sig%20%7B%20abstract.params%28foo%3A%20Integer%2C%20bar%3A%20String%29.void%20%7D%0A%20%20def%20initialize%28foo%2C%20bar%29%0A%20%20end%0Aend%0A%0Aclass%20Bar%20%3C%20Foo%0A%20%20def%20initialize%28foo%2C%20bar%29%0A%20%20%20%20super%28%29%0A%20%20end%0Aend%0A%0ABar.new%281%2C%20%22foo%22%29) ```ruby # typed: true require "sorbet-runtime" class Foo extend T::Sig extend T::Helpers abstract! sig { abstract.params(foo: Integer, bar: String).void } def initialize(foo,...
In https://github.com/shopify/ruby-style-guide#syntax we say: > omit parentheses when the invocation is single-line and the method [...] is a class method call with implicit receiver. But currently this isn't enforced by...
I've just upgraded to the latest gem so ran into this. The `Capybara/FeatureMethods` cop suggests that `scenario 'with OAuth'` should be changed to `it 'with OAuth'`. I feel this is...
I wonder if it would be feasible to checking for stubbing the SUT even when `subject` isn't used, e.g.: ``` it "..." do loan = Foo.new allow(loan).to receive(:bar).and_return(:baz) expect(loan.bar).to eq(:baz)...
(following up from internal Slack discussion) This rule is [disabled](https://github.com/rubocop/rubocop/blob/5af4bad2b420a2c7d4bd2d5385191cadda2f5586/config/default.yml#L4152) by default in RuboCop's config. It was enabled in https://github.com/Shopify/ruby-style-guide/pull/486. It's causing some problems since it can contradict `Style/NegatedIf`. Also...
From the suggestion in https://github.com/rubyapi/rubyapi/pull/1417#issuecomment-1451696255 ~I think `current` would be clearer though. `latest` might be interpreted as the latest commit on `master`.~ Closes https://github.com/rubyapi/rubyapi/issues/1215
Closes https://github.com/rubyapi/rubyapi/issues/1215
Running `bundle` locally on main is causing changes to `Gemfile.lock`: ``` spoom main % g diff diff --git a/Gemfile.lock b/Gemfile.lock index d0a6850..2cf1a68 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -40,8 +40,8...
Minor nit: ``` Duplicated RBI for ::RubyVM::YJIT::enabled?: * https://github.com/sorbet/sorbet/tree/master/rbi/core/ruby_vm.rbi#L804 * sorbet/rbi/shims/core/ruby_vm.rbi:5:4-5:26 ``` Should we output the second entry as `sorbet/rbi/shims/core/ruby_vm.rb#5`? I don't think the column numbers are needed.
Tapioca [skips particular gems](https://github.com/Shopify/tapioca/blob/559f365e825e9160896c14041fcef1fc6cbc67c2/lib/tapioca/gemfile.rb#L155-L165), which can be confusing. I ran into this when trying to add RBIs for `debug`. ``` % tapioca gem debug Requiring all gems to prepare for...