Page MenuHomePhabricator

Fix linting for spacing around default parameter assignment in function/method declaration
ClosedPublic

Authored by joshuaspence on Sep 6 2014, 1:22 AM.
Tags
None
Referenced Files
F14044789: D10429.diff
Tue, Nov 12, 9:14 PM
F14043124: D10429.id25089.diff
Tue, Nov 12, 7:43 AM
F14037812: D10429.diff
Sun, Nov 10, 7:19 PM
F14022745: D10429.diff
Wed, Nov 6, 7:08 PM
F14017717: D10429.id25089.diff
Mon, Nov 4, 9:41 PM
F13977725: D10429.id25157.diff
Fri, Oct 18, 7:46 PM
F13977224: D10429.id25089.diff
Fri, Oct 18, 5:42 PM
F13971905: D10429.id25157.diff
Thu, Oct 17, 4:01 PM
Subscribers

Details

Summary

Fix linting for spacing around default parameter assignment in function/method declaration so that function foo($x = null) {} is preferred in favor of function foo($x=null) {}.

Test Plan

Modified existing unit tests.

Diff Detail

Repository
rARC Arcanist
Branch
paramspace
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2447
Build 2451: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Fix linting for spacing around default parameter assignment in function/method declaration.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Sep 8 2014, 2:34 PM