Closed rul closed 1 week ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: rul Once this PR has been reviewed and has the lgtm label, please assign nan-yu for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
just a couple picks
Another pass might be to use local
for all variables defined inside functions and to convert them all to common name style (lowercase for locals)
Another pass might be to use
local
for all variables defined inside functions and to convert them all to common name style (lowercase for locals)
Makes sense. I think it's a bit out of scope for #801, so I've created #903.
This contribution shellchecks the whole
test_e2e.sh
file. Check individual commits for more details. I've ranmake test
and verified that all tests still pass. Part of #891.