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

	Unknown Object (File)
	Wed, Apr 30, 04:45

	Unknown Object (File)
	Mon, Apr 28, 14:18

	Unknown Object (File)
	Fri, Apr 25, 16:16

	Unknown Object (File)
	Thu, Apr 24, 07:09

	Unknown Object (File)
	Wed, Apr 23, 10:15

	Unknown Object (File)
	Mon, Apr 21, 01:30

	Unknown Object (File)
	Sun, Apr 20, 21:55

	Unknown Object (File)
	Sun, Apr 20, 21:12

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