Piyush Aggarwal
Piyush Aggarwal
@jleveque: Can you please review the changes?
@jleveque: From what I understood, I modified the code to meet the requirements. Can you please check if the update suffices all the requirements? If not, please let me know...
@jleveque: Sorry for pestering you again but this is my first contribution to the open-source community. Hope you understand! I'm not able to understand what methods are you referring to...
@jleveque: The update consists of two changes. 1. Reverting the `read_eeprom_bytes()` return statement back to as it was. 2. Using more descriptive names of variables. The error states - ```...
Hi, I'd like to take this issue. It's a requirement of my course and therefore, I'll make sure to resolve it. Can you please assign it to me?
@jleveque: I went through the 'eeprom_base' file but I'm not sure if I understand the issue completely. Can you please explain the issue a bit more or provide me with...
@jleveque: Hi. Are there any test scripts available that I can run to verify the changes made? There are no instructions in the contributing guide regarding tests.
@jleveque: Can you please guide me on how to use it? This is new for me. Thanks!
@jleveque: I'm afraid I don't have access to SONiC device. Can this utility test be performed online or on a virtual machine using Windows machine and qemu or any other...
@jleveque: I have created a Pull Request and it has passed all the checks. Can you please let me know how to propagate with the review process?