Paths

Table of Contentst

Follow PEP-8 style guide
ClosedPublic
Actions

Authored by dereckson on Jan 27 2016, 15:56.

Tags

None

Referenced Files

	F8004811: D265.diff
	Tue, May 6, 14:35

	F7950787: D265.id618.diff
	Mon, May 5, 18:32

	Unknown Object (File)
	Sat, May 3, 09:39

	Unknown Object (File)
	Fri, May 2, 15:11

	Unknown Object (File)
	Fri, May 2, 06:04

	Unknown Object (File)
	Thu, May 1, 06:27

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

	Unknown Object (File)
	Thu, Apr 24, 00:10

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rNOTIFCLI25b3a46a4d39: Follow PEP-8 style guide

Summary

Two blank lines to separate functions
No space before functions' arguments declaration
Remove two semicolons, append a space after a loose #comment
Break some > 79 characters line

Test Plan

arc lint

Diff Detail

Repository

rNOTIFCLI Notifications center CLI client

Lint

Lint Passed

Unit

No Test Coverage

Branch

pep8

Build Status

Buildable 334
Build 384: arc lint + arc unit

Event Timeline

dereckson updated this revision to Diff 617.Jan 27 2016, 15:56

dereckson retitled this revision from to Follow PEP-8 style guide.

dereckson updated this object.

dereckson edited the test plan for this revision. (Show Details)

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Jan 27 2016, 15:58

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jan 27 2016, 15:58

dereckson updated this object.Jan 27 2016, 15:58

dereckson edited edge metadata.

dereckson updated this object.

Closed by commit rNOTIFCLI25b3a46a4d39: Follow PEP-8 style guide (authored by dereckson, committed by dereckson). · Explain WhyJan 27 2016, 15:59

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

notifications

61 lines

Diff 617

View Options

Follow PEP-8 style guideClosedPublicActions