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)
	Mon, Oct 20, 16:11

	Unknown Object (File)
	Mon, Oct 20, 16:11

	Unknown Object (File)
	Mon, Oct 20, 14:24

	Unknown Object (File)
	Sun, Oct 19, 18:40

	Unknown Object (File)
	Sun, Oct 19, 02:09

	Unknown Object (File)
	Sat, Oct 18, 12:39

	Unknown Object (File)
	Sat, Oct 18, 09:15

	Unknown Object (File)
	Tue, Oct 14, 18:05

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