Support Ruby 3.1 hash shorthand syntax

[presidentbeef]

like { x: }

(I guess at this point ruby2ruby is pretty far behind on Ruby syntax...)

[zenspider]

I'm adding a positive test case cribbed off your negative test case:

  def test_hash_shorthand
    inn = s(:hash, s(:lit, :k), nil)
    out = '{ k: }'

    assert_parse inn, out
  end

[zenspider]

(I guess at this point ruby2ruby is pretty far behind on Ruby syntax...)

It is... Most of the tests come in via pt_testcase.rb from sexp_processor. I have not bothered porting new tests from ruby_parser to pt_testcase.rb and that's caused some slippage.

2.7 is probably covered pretty well (tho, maybe not **kws?)... but the 3.0+ tests could use an audit.

[zenspider]

Added! Thanks!

[presidentbeef]

Huh, I have no idea where my positive test went. I know I wrote it and I wrote it before I wrote what ended up in this PR. Sorry about that!

Thanks!

[zenspider]

No worries!

…

On Apr 15, 2022, at 13:03, Justin Collins ***@***.***> wrote:  Huh, I have no idea where my positive test went. I know I wrote it and I wrote it before I wrote what ended up in this PR. Sorry about that! Thanks! — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you modified the open/close state.

[zenspider]

What do you think about going back to the old version but provide a defensive one via en env var or something?

…

On Apr 15, 2022, at 13:03, Justin Collins ***@***.***> wrote:  Huh, I have no idea where my positive test went. I know I wrote it and I wrote it before I wrote what ended up in this PR. Sorry about that! Thanks! — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you modified the open/close state.

[presidentbeef]

@zenspider I think that last comment was for seattlerb/sexp_processor#30?