Paths

Table of Contentst

Avoid both assignment and comparison in same time
ClosedPublic
Actions

Authored by dereckson on Mar 29 2018, 21:34.

Tags

None

Referenced Files

	F24176104: D1538.diff
	Wed, Feb 11, 01:50

	F24176049: D1538.id3927.diff
	Wed, Feb 11, 01:49

	F24172492: D1538.diff
	Wed, Feb 11, 00:23

	Unknown Object (File)
	Fri, Feb 6, 22:24

	Unknown Object (File)
	Thu, Feb 5, 20:09

	Unknown Object (File)
	Thu, Feb 5, 19:26

	Unknown Object (File)
	Thu, Feb 5, 19:15

	Unknown Object (File)
	Thu, Feb 5, 07:55

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rNOTIF0575d2497ed6: Avoid both assignment and comparison in same time

Test Plan

phpcs with Generic.CodeAnalysis.AssignmentInCondition rule

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Mar 29 2018, 21:34

dereckson created this revision.

Harbormaster failed remote builds in B2412: Diff 3924!Mar 29 2018, 21:34

Not so no op

Rebase

Harbormaster completed remote builds in B2414: Diff 3927.Mar 29 2018, 21:55

dereckson accepted this revision.Mar 29 2018, 21:55

This revision is now accepted and ready to land.Mar 29 2018, 21:55

Closed by commit rNOTIF0575d2497ed6: Avoid both assignment and comparison in same time (authored by dereckson). · Explain WhyMar 29 2018, 21:56

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Analyzers/

GitHub/

Events/

RepositoryEvent.php

3 lines

Diff 3928

View Options

app/Analyzers/GitHub/Events/RepositoryEvent.php

Avoid both assignment and comparison in same timeClosedPublicActions