Page MenuHomeDevCentral

Ensure lines are at most 80 characters
ClosedPublic

Authored by dereckson on Mar 30 2018, 00:00.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 02:35
Unknown Object (File)
Sat, Dec 7, 18:16
Unknown Object (File)
Mon, Dec 2, 10:13
Unknown Object (File)
Nov 17 2024, 05:21
Unknown Object (File)
Nov 17 2024, 04:46
Unknown Object (File)
Nov 17 2024, 04:43
Unknown Object (File)
Nov 17 2024, 03:03
Unknown Object (File)
Nov 16 2024, 23:35
Subscribers
None

Details

Summary

Several strategies have been used:

  • group namespaces use declarations
  • heredoc and newdoc strings
  • prune non informative comments or switch them to PHP 7 type hints
  • break lines

References:

Test Plan

phpcs with Generic.Files.LineLength sniff

Diff Detail

Repository
rNOTIF Notifications center
Lint
Lint Passed
Unit
Tests Passed
Branch
wrap-long-lines (branched from master)
Build Status
Buildable 2426
Build 2674: arc lint + arc unit

Event Timeline

dereckson created this revision.
dereckson added a reviewer: dereckson.
dereckson edited the summary of this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)

Don't touch porcelain Phabricator non refactored code, tests and code don't agree on return types

This revision is now accepted and ready to land.Mar 30 2018, 00:41
This revision was automatically updated to reflect the committed changes.