Page MenuHomeDevCentral
Differential D2577

Don't shadow variables
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 20:14.
Tags
None
Referenced Files
F10830644: D2577.id.diff
Mon, Jul 28, 05:16
F10830305: D2577.id6511.diff
Mon, Jul 28, 04:57
F10826305: D2577.diff
Mon, Jul 28, 01:17
F10815981: D2577.id6510.diff
Sun, Jul 27, 15:20
Unknown Object (File)
Sun, Jul 27, 04:46
Unknown Object (File)
Fri, Jul 25, 14:41
Unknown Object (File)
Sat, Jul 19, 09:36
Unknown Object (File)
Thu, Jul 17, 18:07
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