Fix redefined-outer-name issue
ClosedPublic
Actions

Authored by dereckson on Feb 21 2016, 18:18.

Tags

None

Referenced Files

	F53973186: D295.diff
	Tue, Oct 13, 21:03

	F53943984: D295.id695.diff
	Tue, Oct 13, 12:06

	F53930507: D295.id694.diff
	Tue, Oct 13, 09:13

	F53916039: D295.id695.diff
	Tue, Oct 13, 06:23

	Unknown Object (File)
	Sun, Oct 11, 21:36

	Unknown Object (File)
	Fri, Oct 9, 10:16

	Unknown Object (File)
	Fri, Oct 9, 02:33

	Unknown Object (File)
	Thu, Oct 8, 23:00

View All Files

Subscribers

None

Details

Reviewers

dereckson

Maniphest Tasks

T756: CLI client: Redefining name 'app' from outer scope (line 180) (redefined-outer-name)

Commits

rNOTIFCLI79e2308c2c93: Fix redefined-outer-name issue

Summary

Fixes T756.

Test Plan

pylint notifications

Diff Detail

Repository

rNOTIFCLI Notifications center CLI client

Lint

Lint Passed

Unit

No Test Coverage

Branch

redefined-outer-name

Build Status

Buildable 382
Build 459: arc lint + arc unit

Event Timeline

dereckson updated this revision to Diff 694.Feb 21 2016, 18:18

dereckson retitled this revision from to Fix redefined-outer-name issue.

dereckson updated this object.

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

dereckson added a reviewer: dereckson.

dereckson added a task: T756: CLI client: Redefining name 'app' from outer scope (line 180) (redefined-outer-name).

dereckson accepted this revision.Feb 21 2016, 18:20

dereckson edited edge metadata.

This revision is now accepted and ready to land.Feb 21 2016, 18:20

Closed by commit rNOTIFCLI79e2308c2c93: Fix redefined-outer-name issue (authored by dereckson, committed by dereckson). · Explain WhyFeb 21 2016, 18:21

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

notifications

14 lines

Diff 694

View Options

Fix redefined-outer-name issueClosedPublicActions