Firmware, i command is causing a reboot

[nk25719]

Describe the bug
A clear and concise description of what the bug is.
Short Summary of the expected behavior:
Short Summary of the buggy behavior:
the i command is returning the info of the board but then forcing a reboot.

[TheBakedPotato]

Alright. Found the problem. Mea culpa. Line 432 in GPAD_HAL.cpp. Essentially by using the enum type Command it can't properly/safely be formatted using the F macro. The F macro expects a string literal, like a char. I overlooked this when I implemented the changes for #206 . It will be an easy fix. Just gotta implemented a method for turning the Command type into a char. 🤷

[TheBakedPotato]

So, until a PR is merged, which will hopefully be very soon, you can simply comment out that line for now and everything will work.

This is simply a note for interest sake. The runtime error that was being experienced would have actually been caught in a language like Rust by the compiler. Honestly, I will fully own up to me probably being a lazy and not fully testing it because I was so used to Rust. I will try to be more diligent about expected runtime errors!

The take away here is that C/C++ will very VERY happily do some implicit conversions between types. This should absolutely be avoided. If a cast IS required, use the static_cast function. Otherwise, with Implict casts, we end up with runtime errors like what is happening here.