-
Notifications
You must be signed in to change notification settings - Fork 422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test_e2e.sh: use local
for all variables defined inside functions
#903
Comments
Hi @rul, thank you for the contribution, as I can observe the PR is merged now, so you can now close this issue. |
AFAICT, this is not done:
etc |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
Use
local
for all variables defined inside functions; also convert them all to common name style (lowercase for locals)Originally posted by @thockin in #902 (comment)
The text was updated successfully, but these errors were encountered: