-
-
Notifications
You must be signed in to change notification settings - Fork 417
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
Who is this extension for #103
base: master
Are you sure you want to change the base?
Conversation
Thanks for this PR. I'm sorry that this extension didn't help you so far. However, I have to disagree that this extension is not for professional use.
Can you describe your purpose? Maybe I can help you. |
Sure! I'm not doing anything too specific, my use is to use it like I would use a debugger in Golang tests and such, in a codebase that is not supposed to have any debug-visualizer specific code. My problem currently, is that I dont find any convenient way of using the extension, without having specialized functions inside the codebase that return an extension-friendly json format. Plus the code kinda has to revolve around the extension, which is a deal-braker. And from what I've seen in the examples and videos so far, that was the only way to use the extension for me so far.
Please tell me about it. That could actually turn out to be quite helpful! |
I see that you are using go. Indeed, the support for JavaScript based languages is much better. If you have technical ideas how to visualize stuff in go, I'd be happy to help to make this possible with this extension! In the end it boils down to the watch window displaying some compatible json string. Currently, pre/post-processing is hardcoded in the extension and only for JavaScript non-trivial pre/post-processing is done. |
@hediet From what I see there is a generic and a js evaluation engine, where the JS I assume is supposed to use the JS data extraction engine. I'll study the codebase better and see how I can help. For now I guess I should be closing this PR. But I guess its still relevant for languages like Golang. |
Yes!
I don't want to merge the part that states that this extension is not for professional use. But you are right, it could be stated that this extension is difficult to use with golang and other compiled languages. |
Changed from professional use, to precompiled languages.
I changed the description to describe the difficulties with pre-compiled languages. Feel free to give any feedback :) Edit: Even if it's the way of expression, or if you believe there's excessive information, or something that doesn't stick to this project's profile, please let me know. |
Hey, for first amazing work! Great job!
I just run into a small time investment issue when I wanted to tried to integrate this extension into my workflow. I had no knowledge of how to use the extension, nor the description makes it quite clear to be fair. :P
After spending some time, I found out that the extension requires you to write code that is dedicated to this extension, instead of going vica versa. I found out about this after I started using the extension and spending some time with it, because I simply had no idea how it works. So, I concluded that it wouldn't be of use at the current time for the purpose I wanted it.
The point is, that I've realized this extension wasn't for me after I've spend some hours with it. We could improve this with some description on the README.
Please feel free to improve the description, or any grammar mistakes I might have done! :) Great work once again.