Page MenuHomeDevCentral
Differential D2577

Don't shadow variables
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 20:14.
Tags
None
Referenced Files
F35259444: D2577.diff
Wed, Jul 8, 11:44
F35194163: D2577.diff
Tue, Jul 7, 16:08
Unknown Object (File)
Sun, Jul 5, 15:16
Unknown Object (File)
Sat, Jul 4, 20:42
Unknown Object (File)
Fri, Jul 3, 21:47
Unknown Object (File)
Fri, Jul 3, 20:48
Unknown Object (File)
Thu, Jul 2, 21:58
Unknown Object (File)
Thu, Jul 2, 16:27
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rOPS5a1d179f76e5: Don't shadow variables
Summary

Variables like id and list are built-ins.

A method argument can't share the same name with the procedural code.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Errors
SeverityLocationCodeMessage
Errorutils/dump-py-state.py:66E501flake8 E501
Advice_modules/opensearch.py:24F821flake8 F821
Advice_modules/opensearch.py:66F821flake8 F821
Advice_modules/opensearch.py:86F821flake8 F821
Unit
No Test Coverage
Branch
main
Build Status
Buildable 4038
Build 4290: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 7 2022, 20:14
dereckson created this revision.
Harbormaster completed remote builds in B4038: Diff 6510.Mar 7 2022, 20:14
dereckson accepted this revision.Mar 7 2022, 20:16
This revision is now accepted and ready to land.Mar 7 2022, 20:16
This revision was landed with ongoing or failed builds.Mar 7 2022, 20:16
Closed by commit rOPS5a1d179f76e5: Don't shadow variables (authored by dereckson). · Explain Why
This revision was automatically updated to reflect the committed changes.
dereckson added a commit: rOPS5a1d179f76e5: Don't shadow variables.

Revision Contents
Changeset List

PathSize
_modules/
4 lines
44 lines
6 lines
roles/
paas-jails/
jails/
2 lines
utils/
4 lines
8 lines
4 lines

Diff 6510

View Options

_modules/jails.py

Loading...
View Options

_modules/opensearch.py

Loading...
View Options

_modules/zr.py

Loading...
View Options

roles/paas-jails/jails/init.sls

Loading...
View Options

utils/dump-py-state.py

Loading...
View Options

utils/generate-webcontent-index.py

Loading...
View Options

utils/next-uid.py

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator