Page MenuHomeDevCentral
Differential D2577

Don't shadow variables
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 20:14.
Tags
None
Referenced Files
F3614497: D2577.id.diff
Wed, Oct 2, 09:14
F3614484: D2577.id.diff
Wed, Oct 2, 09:05
Unknown Object (File)
Wed, Sep 18, 11:08
Unknown Object (File)
Tue, Sep 17, 22:29
Unknown Object (File)
Sat, Sep 7, 12:27
Unknown Object (File)
Fri, Sep 6, 00:27
Unknown Object (File)
Wed, Sep 4, 12:59
Unknown Object (File)
Sep 1 2024, 16:28
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