Robert Mosolgo
Robert Mosolgo
Thanks for sharing what you found here. I don't have plans to work further on this, so I'm going to close the issue. If anyone runs into similar trouble, please...
I'm open to adding some formal structure for this, but I don't have plans to pursue it myself, so I'm going to close this out.
Thanks for taking a look. I also just saw your answer over at https://bugs.ruby-lang.org/issues/20081 (thanks for that, too 😅 ). That `ensure` suggestion makes this script work, too: ```diff +...
Thanks for the detailed report -- I agree that it should work like you described but instead, I seem to have a spec with the _opposite_: https://github.com/rmosolgo/graphql-ruby/blob/master/spec/graphql/schema/build_from_definition_spec.rb#L15-L35 🤦 I'm open...
I think you're right, it looks like this one is used instead: https://github.com/rmosolgo/graphql-ruby/blob/f72c7a18e9c0f676fdf1087c3d5208b90752e1be/lib/graphql/static_validation/base_visitor.rb#L190 D'oh, but I just checked GraphQL-Pro, it's still using this as part of the OperationStore feature. I'm...
Hi, sure thing, thanks for taking a look! Here's a benchmark I wrote to demonstrate the impact: ```ruby require "bundler/inline" gemfile do gem "prawn" gem "matrix" gem "benchmark-ips" gem "memory_profiler"...
Yeah, that sounds good -- looking into the `prawn` source, it seems like `pairs` is only used when `options[:kerning]` is used by the application. So if we skipped initializing the...
Now I see that at least _some_ compatibility was restored (https://github.com/rubygems/rubygems/pull/5826), but for some reason, in my case, it's still doing this full download. Is there something else I can...
Cool, thanks for sharing your thoughts on it. I'll size up my options next week and follow up back here if I take a crack at updating that rake task.
Thanks for diving into that! I'd love to use it, I'll keep my eyes on the linked PR. For my own part, I started looking into the source code and...