Fix redefined-outer-name issue
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13461890: D295.id694.diff
	Sat, Nov 22, 13:19

	F13459367: D295.diff
	Sat, Nov 22, 12:49

	F13459284: D295.id695.diff
	Sat, Nov 22, 12:48

	F13358128: D295.id694.diff
	Fri, Nov 21, 19:21

	Unknown Object (File)
	Mon, Nov 17, 17:12

	Unknown Object (File)
	Mon, Nov 17, 12:56

	Unknown Object (File)
	Fri, Nov 14, 20:37

	Unknown Object (File)
	Mon, Nov 10, 00:58

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 Not Applicable

Unit

Tests Not Applicable

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 695

View Options

Fix redefined-outer-name issueClosedPublicActions