Paths

Table of Contentst

Enforce opening brace is on same line than method declaration
ClosedPublic
Actions

Authored by dereckson on Mar 30 2018, 00:52.

Tags

None

Referenced Files

	F24689793: D1547.id3949.diff
	Thu, Mar 5, 12:05

	F24689606: D1547.id3949.diff
	Thu, Mar 5, 11:50

	F24689604: D1547.id3949.diff
	Thu, Mar 5, 11:49

	F24687830: D1547.id3948.diff
	Thu, Mar 5, 09:34

	F24687662: D1547.id3948.diff
	Thu, Mar 5, 09:18

	F24684119: D1547.diff
	Thu, Mar 5, 04:01

	Unknown Object (File)
	Tue, Mar 3, 14:56

	Unknown Object (File)
	Mon, Mar 2, 22:19

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rNOTIF60f393712423: Enforce opening brace is on same line than method declaration

Summary

Nasqueron uses K&R style, Laravel not, this change standardizes
the Laravel-generated code.

This is a follow-up for 85fc0f62b8e1 (same change for classes).

Test Plan

phpcs with Generic.Functions.OpeningFunctionBraceKernighanRitchie sniff

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Passed

Unit

Tests Passed

Branch

OpeningFunctionBraceKernighanRitchieSniff

Build Status

Buildable 2427
Build 2675: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 30 2018, 00:52

dereckson created this revision.

Harbormaster completed remote builds in B2427: Diff 3948.Mar 30 2018, 00:52

dereckson accepted this revision.Mar 30 2018, 00:53

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

This revision is now accepted and ready to land.Mar 30 2018, 00:53

Closed by commit rNOTIF60f393712423: Enforce opening brace is on same line than method declaration (authored by dereckson). · Explain WhyMar 30 2018, 00:53

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

app/

Providers/

AppServiceProvider.php

6 lines

RouteServiceProvider.php

6 lines

Diff 3948

View Options

app/Providers/AppServiceProvider.php