-
Notifications
You must be signed in to change notification settings - Fork 110
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
feat: remove string checks and add robust checks for fileType #658
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Jougan-0 <[email protected]>
Tested for these 4 files
|
Meshery Design
|
K8s manifest
|
Helm chart of meshery |
Used this script locally to check these
Here is the result
|
I looked at the failing tests these are due to 3 packages
The unit tests for codecov they are failing as they have fragile test cases written. |
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.
Whoa! Very cool. 😎 👏 @Jougan-0, this is quite timely as this particular area is being actively discussed - #657
// @aabidsofi19
Description
This PR fixes issue of fragile introspecing logic for file types.
Tested locally for all the file types.
Notes for Reviewers
Signed commits