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
Unknown Object (File)
Sat, Nov 23, 5:40 AM
Unknown Object (File)
Fri, Nov 22, 8:50 PM
Unknown Object (File)
Tue, Nov 19, 3:56 PM
Unknown Object (File)
Fri, Nov 15, 1:54 PM
Unknown Object (File)
Tue, Nov 12, 9:14 PM
Unknown Object (File)
Tue, Nov 12, 7:43 AM
Unknown Object (File)
Sun, Nov 10, 7:19 PM
Unknown Object (File)
Wed, Nov 6, 7:08 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