Skip to content
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.

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

Add test for unassigned required field in constructor #84

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

calummoore
Copy link
Contributor

No description provided.

@linear
Copy link

linear bot commented Apr 4, 2023

@calummoore calummoore requested a review from mateuszmlc April 4, 2023 19:36
@calummoore
Copy link
Contributor Author

It looks like this already creates an error, I added a test for this. Ideally, we wouldn't compile code like this, but we'll have to make do for now with this.

@calummoore calummoore merged commit ab97687 into main Apr 7, 2023
@calummoore calummoore deleted the eng-538-required-array-should-error-if-not branch June 6, 2023 22:28
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