Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Ensure non-null record field are mandatory in arguments #127

Closed
Totktonada opened this issue Apr 20, 2018 · 1 comment
Closed

Ensure non-null record field are mandatory in arguments #127

Totktonada opened this issue Apr 20, 2018 · 1 comment
Labels
bug Something isn't working

Comments

@Totktonada
Copy link
Member

There is the test for compound offset which pass validation and fails in runtime. It must not pass validation.
@Totktonada Totktonada added the bug Something isn't working label Apr 20, 2018
@Totktonada Totktonada mentioned this issue Apr 20, 2018
Merged
@Totktonada
Copy link
Member Author

Fixed with PR #180. This check still performed in the execution time, while coercing values. It would be good to have compile time validation for immediate arguments, but this is subject to a separate issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Totktonada