Paths

Table of Contentst

Rename jails.list into jails.list_jails
ClosedPublic
Actions

Authored by dereckson on Mar 7 2022, 22:13.

Tags

None

Referenced Files

	F53696838: D2580.diff
	Sun, Oct 11, 15:13

	F53693193: D2580.id6516.diff
	Sun, Oct 11, 14:41

	F53621521: D2580.diff
	Sun, Oct 11, 05:20

	F53568255: D2580.id.diff
	Sat, Oct 10, 22:15

	Unknown Object (File)
	Fri, Oct 9, 13:56

	Unknown Object (File)
	Fri, Oct 9, 13:15

	Unknown Object (File)
	Fri, Oct 9, 08:40

	Unknown Object (File)
	Thu, Oct 8, 21:56

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rOPSe89c73d4163b: Rename jails.list into jails.list_jails

Summary

list() is a reserved method name in Python,
so it shadows it.

This is a follow-up for 5a1d179f76e5.

Test Plan

make test

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-shadow-jails

Build Status

Buildable 4041
Build 4293: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 7 2022, 22:13

dereckson created this revision.

Harbormaster completed remote builds in B4041: Diff 6516.Mar 7 2022, 22:13

dereckson accepted this revision.Mar 7 2022, 22:13

This revision is now accepted and ready to land.Mar 7 2022, 22:13

Closed by commit rOPSe89c73d4163b: Rename jails.list into jails.list_jails (authored by dereckson). · Explain WhyMar 7 2022, 22:14

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rOPSe89c73d4163b: Rename jails.list into jails.list_jails.

Revision Contents
Changeset List

Path

Size

_tests/

modules/

test_jails.py

6 lines

Diff 6516

View Options

_tests/modules/test_jails.py

Rename jails.list into jails.list_jailsClosedPublicActions