Improve check-letsencrypt-certificates Python code
ClosedPublic
Actions

Authored by dereckson on Sep 9 2018, 08:23.

Tags

None

Referenced Files

	F10156505: D1697.diff
	Sat, Jun 21, 01:25

	Unknown Object (File)
	Fri, Jun 13, 17:02

	Unknown Object (File)
	Thu, Jun 12, 00:40

	Unknown Object (File)
	Wed, Jun 4, 03:49

	Unknown Object (File)
	May 9 2025, 20:17

	Unknown Object (File)
	May 6 2025, 13:19

	Unknown Object (File)
	May 6 2025, 09:07

	Unknown Object (File)
	May 6 2025, 09:01

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rOPS2666706e3aad: Improve check-letsencrypt-certificates Python code

Summary

port to Python 3
use snake_case for method names and local variables
use _ instead of an unused variable name

Test Plan

flake8

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

improve-python (branched from master)

Build Status

Buildable 2649
Build 2897: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Sep 9 2018, 08:23

dereckson created this revision.

Harbormaster completed remote builds in B2649: Diff 4317.Sep 9 2018, 08:23

dereckson accepted this revision.Sep 9 2018, 08:25

This revision is now accepted and ready to land.Sep 9 2018, 08:25

Closed by commit rOPS2666706e3aad: Improve check-letsencrypt-certificates Python code (authored by dereckson). · Explain WhySep 9 2018, 08:25

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

scripts/

byTasks/

HTTP/

check-letsencrypt-certificates

53 lines

Diff 4317

View Options

scripts/byTasks/HTTP/check-letsencrypt-certificates

Improve check-letsencrypt-certificates Python codeClosedPublicActions