-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Set source-type flag required #11020
Set source-type flag required #11020
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #11020 +/- ##
==========================================
+ Coverage 9.00% 9.13% +0.12%
==========================================
Files 146 146
Lines 19274 19283 +9
==========================================
+ Hits 1736 1761 +25
+ Misses 17236 17220 -16
Partials 302 302
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@Yashsharma1911 @MUzairS15 does Meshery Server still not auto-detect design source type? |
@lekaf974 please don’t assign new error code value manually. Just define the new error and the utility will take care of it. |
Will you address the lint checks? |
ba25ce8
to
878a880
Compare
updated but still having issue in file not related to this PR pkg/utils/model.go:119:27: Error return value of `mcv.UpdateModelDefinition` is not checked (errcheck)
mcv.UpdateModelDefinition(&model) |
8a87a61
to
f38d092
Compare
Signed-off-by: mevrin <matthieu.evrin@gmail.com>
Signed-off-by: Mohd Uzair <muzair.shaikh810@gmail.com>
Signed-off-by: Mohd Uzair <muzair.shaikh810@gmail.com>
Signed-off-by: mevrin <matthieu.evrin@gmail.com>
f38d092
to
e43c10e
Compare
Signed-off-by: mevrin <matthieu.evrin@gmail.com>
thanks @lekaf974 |
Notes for Reviewers
This PR fixes #10960
Signed commits
Due to non provided source type, the server returned the error. To avoid this behavior the -s/--source-type flag is now required
new behavior