Skip to content
This repository has been archived by the owner on Oct 17, 2019. It is now read-only.

Support "value is type $primitive" for constraining choices #31

Merged
merged 2 commits into from
Dec 12, 2017

Conversation

cmoesel
Copy link
Member

@cmoesel cmoesel commented Dec 12, 2017

The "value is type" type constraint can also be used to constrain a choice to a subset. In this case, the type you are constraining could be a primitive. Prior to this fix, the parser choked if trying to constrain to a primitive.

Fixes standardhealth/shr-grammar#27.

The "value is type" type constraint can also be used to constrain a choice to a subset.  In this case, the type you are constraining could be a primitive.  Prior to this fix, the parser choked if trying to constrain to a primitive.

Fixes standardhealth/shr-grammar#27.
@cmoesel cmoesel requested a review from Abhijay December 12, 2017 18:34
Copy link
Contributor

@Abhijay Abhijay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good. Unit tests and personal testing both pass.

@Abhijay Abhijay merged commit 24707a9 into master Dec 12, 2017
@Abhijay Abhijay deleted the fix_grammar_27 branch December 12, 2017 18:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot constrain value choice to primitive (from a field)
2 participants