Skip to content

Fix match-pattern-p with logical operator unparsing#401

Merged
mbj merged 1 commit intombj:mainfrom
viralpraxis:fix-match-pattern-p-with-logical-operator
May 14, 2025
Merged

Fix match-pattern-p with logical operator unparsing#401
mbj merged 1 commit intombj:mainfrom
viralpraxis:fix-match-pattern-p-with-logical-operator

Conversation

@viralpraxis
Copy link
Contributor

@viralpraxis viralpraxis commented May 14, 2025

Extracted from #387

$ bundle exec bin/unparser -e 'a in b, and c'
warning: parser/current is loading parser/ruby34, which recognizes 3.4.0-dev-compliant syntax, but you are running 3.4.3.
Please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
(string)
Original-Source:
a in b, and c
Generated-Source:
a in [b, ] && c
Original-Node:
(and
  (match-pattern-p
    (send nil :a)
    (array-pattern-with-tail
      (match-var :b)))
  (send nil :c))
Generated-Node:

```
$ bundle exec bin/unparser -e 'a in b, and c'
warning: parser/current is loading parser/ruby34, which recognizes 3.4.0-dev-compliant syntax, but you are running 3.4.3.
Please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
(string)
Original-Source:
a in b, and c
Generated-Source:
a in [b, ] && c
Original-Node:
(and
  (match-pattern-p
    (send nil :a)
    (array-pattern-with-tail
      (match-var :b)))
  (send nil :c))
Generated-Node:
```
@mbj mbj merged commit 4cca58e into mbj:main May 14, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants