Refactor Language::Nodes::AbstractNode#initialize

[rmosolgo]

It uses keyword arguments which requires Ruby to allocate a Hash. (This is only for .new -> #initialize.) See #4718 (comment)

[tenderlove]

If I can get this and this landed in Ruby 3.4, it will eliminate the hash allocation. If you like the kwargs, keep them! 😄

[rmosolgo]

Wow, that would be great. I'll keep an eye on those PRs, thanks!

[rmosolgo]

One of the two linked PRs to Ruby is merged... 🤞

[rmosolgo]

👋 Hey @tenderlove, are you still considering that PR to do Class#new in Ruby? I'm gearing up for GraphQL-Ruby 3.0 and I'll roll this breaking change to #initialize if necessary.

[tenderlove]

👋 Hey @tenderlove, are you still considering that PR to do Class#new in Ruby? I'm gearing up for GraphQL-Ruby 3.0 and I'll roll this breaking change to #initialize if necessary.

I am, but there's been some blockers so I don't think I'll be able to get this merged until next year maybe. 😭

Please do what you think is best for GraphQL-Ruby and don't wait on me. I think we'll eventually get Ruby changed upstream, but it's going to take more time.