-
Notifications
You must be signed in to change notification settings - Fork 75
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
Test coverage for entrypoint
variable of mlcube deployment
#733
Comments
Stale issue message |
Hey @hasan7n could you please take a look at this? |
Stale issue message |
Hey @hasan7n can you take a crack at this? |
@VukW this should be fixed already with the new API branch, right? |
@sarthakpati No, this part was never touched in new API branch. To tell truth I don't get quite well even after reading parent PR what exactly is the issue about? What do we need to check? |
Ah okay. It is basically that these lines are not getting covered by tests: And since this is (IMO) an integral part of the deploy mechanism, I would assume a test would need to updated/added to cover it. I was trying to ask clarification from @hasan7n but he hasn't responded this thread, yet. |
Are you talking about just a coverage of this line in |
The coverage. |
Originally posted by @sarthakpati in #730 (comment)
The text was updated successfully, but these errors were encountered: