Skip to content

Fix selector operators #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
garyb merged 1 commit into from
Jul 8, 2018
Merged

Fix selector operators #78

garyb merged 1 commit into from
Jul 8, 2018

Conversation

@vyorkin
Copy link
Collaborator

@vyorkin vyorkin commented Jul 1, 2018 •
edited
Loading

Fixes typo here.

Also:

  • Increased the precedence of these selector operators (to make it possible to omit parentheses), but maybe its better to lower the precedence of ? operator. Not sure if this might be a breaking change though
  • Added a couple of tests

resolves #75

@vyorkin vyorkin force-pushed the bugfix/selector-operators branch from 7da0d8a to 0fcfa14 Compare July 1, 2018 16:38
@vyorkin vyorkin force-pushed the bugfix/selector-operators branch from 0fcfa14 to f6d0167 Compare July 1, 2018 16:43
@garyb garyb merged commit f62924f into purescript-contrib:master Jul 8, 2018
@vyorkin vyorkin deleted the bugfix/selector-operators branch July 8, 2018 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

(##) is infix for child instead of with

2 participants

@vyorkin @garyb