-
Notifications
You must be signed in to change notification settings - Fork 83
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
karmor probe
output in json format
#310
Conversation
540aed1
to
d738c34
Compare
d738c34
to
2fb5100
Compare
2fb5100
to
ea6a0c6
Compare
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.
Fix all lint errors.
72f54ae
to
d0be4bc
Compare
3263182
to
6b8f259
Compare
212e7cf
to
087709d
Compare
deprecates #301 |
Signed-off-by: Aryan-sharma11 <aryan1126.sharma@gmail.com>
8be7f6f
to
ea39d08
Compare
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.
I have tested it out, Works great for me. I went through the codebase and realised we are printing a lot of stuff inside the function.
Since the JSON here is providing us with the data in a centralised structure, let's avoid printing data inside.
We could avoid side effects and each function is loosely coupled so as we can use them independently as we need them.
b421180
to
372c01e
Compare
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.
LGTM, a small refactor suggestion
Signed-off-by: Aryan-sharma11 <aryan1126.sharma@gmail.com>
372c01e
to
5d186ec
Compare
Purpose of PR?:
Fixes #302 #298
Does this PR introduce a breaking change?
No
Description
We can get the
karmor probe
data in json format usingkarmor probe -f json
.output for
karmor probe -f text