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
added a better error message and a link to a full explanation #386
added a better error message and a link to a full explanation #386
Conversation
@@ -40,6 +40,12 @@ export class GetTestController extends Controller { | |||
modelsArray: new Array<TestSubModel>(), | |||
numberArray: [1, 2, 3], | |||
numberValue: 1, | |||
object: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are these necessary for the tests to compile?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, unfortunately they are necessary for the tests to compile. It happened because one of the previous PRs got merged without the tests running (woops!). But now that the previous PR added husky, these kind of things will be caught.
@endor I pushed the corrections. Thank you so much for reviewing this so thoroughly. It's wonderful to be collaborating with you. :) |
This fixes #384