leongross
leongross
But can you explain how the command line that was executed was changed? This seems to have the potential to execute arbitrary commands on the runner which in fact could...
I see, thank you for clarifying.
@jhand2 the ROM is now unmodified, all valaues match.
@binjip978
@binjip978
I agree that PR #3106 got quite messy, but I think this PR does it better. Adding the linter is jus `+1` and the other fixes are linter fixes only....
After we finish this, I propose adding [`errcheck`](https://golangci-lint.run/usage/linters/#errcheck). Because this PR also adds error checks that were not here before it makes sense to wait until this is merged to...
@binjip978 @rminnich