Trevor Elliott
Trevor Elliott
We have a policy of testing changes to Sorbet against Stripe's codebase before merging them. I've kicked off a test run for the current PR. When the build finishes, I'll...
Yes, this is due to the need to support multiple possible return types, but sorbet not currently supporting multiple signatures for stdlib methods that use keyword args.
@uweigand, could you take a closer look at my changes to the s390x backend? In particular the changes to the lowering of `bmask`, as they increased the instruction count for...
> Following up on myself, it would be even simpler _and_ better to use the existing `value_nonzero` helper, which will fold in previous comparisons if possible: > > ``` >...
Here's a link to the [2021-08-23](https://github.com/bytecodealliance/meetings/blob/main/cranelift/2021/cranelift-08-23.md#notes-1) meeting notes where this was discussed originally.
There has been discussion recently about whether or not we need the boolean types at all, and we seem to be converging on the opinion that we should remove them...
This is a little more complicated than I first thought, and is unrelated to the keyword arguments refactoring. ``` irb(main):001:1* def foo(a={}, b: nil) irb(main):002:1* p a irb(main):003:1* p b...
Thanks! I think we should change this to look more like `each_with_object`, using `T.untyped` for the type of the block argument, returning a value of the same type as `initial`....
cc @Morriar
Would you mind rebasing this PR? Once that's done, I'm happy to click the merge button :+1: