Suyesh Badge
Suyesh Badge
@aryanmehrotra, @vipul-rawat, and @Umang01-hash, I'm initiating a discussion thread to commence the implementation of the mentioned feature. Let's delve into the various potential changes and methodologies to effectively incorporate this...
Hey @vipul-rawat, I've pushed the requested changes along with the formatting and linting fixes. Please have a look once you get a chance and let me know if anything is...
Certainly, @vipul-rawat will make the changes and update you accordingly.
> I am still not sure if this would handle the pointers You're right to be concerned about handling pointers. The current implementation assumes that the fields are direct values...
Hey @vipul-rawat , I've updated the filtered logging feature to support pointer values and added documentation on how to use it. However, one test check is failing, and I'm not...
Hello @vipul-rawat, Could you kindly provide further clarification regarding the utilization of configuration readings in this context? Specifically, would you prefer me to: 1. Pass the configuration explicitly to the...
Hey @vipul-rawat, I've implemented the changes as per your instructions. Additionally, I addressed several linting issues that were present. Kindly review the updates at your convenience. With the latest implementation,...
Also if possible could you please re-run the checks, some of them are failing.
> @SuyeshBadge - Are you still working on this? Yes @srijan-27 will update soon
Hey @srijan-27 and @vipul-rawat, The changes we discussed have been implemented. Please review them and let me know if anything else is needed. Thanks!