Page MenuHomeDevCentral
Differential D38

PhpFpmEnvironment: get → printConfig
ClosedPublic
Actions

Authored by dereckson on Nov 8 2015, 22:59.
Tags
Referenced Files
F27251978: D38.id84.diff
Sat, May 2, 02:57
F27251962: D38.id79.diff
Sat, May 2, 02:57
F27247475: D38.diff
Fri, May 1, 23:28
Unknown Object (File)
Thu, Apr 30, 17:55
Unknown Object (File)
Thu, Apr 30, 13:09
Unknown Object (File)
Thu, Apr 30, 08:29
Unknown Object (File)
Wed, Apr 29, 09:32
Unknown Object (File)
Tue, Apr 28, 16:34
View All Files
Subscribers
None

Details

Reviewers
Sandlayth
Commits
rDPHPVIIFPMb32adb190b79: PhpFpmEnvironment: get → printConfig
rDPHPFPMb32adb190b79: PhpFpmEnvironment: get → printConfig
Summary

Method printing stuff should be in 'print' not in 'get'.

Test Plan

trivial, no op

Diff Detail

Repository
rDPHPFPM Docker image for Nginx, php-fpm
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
get-print
Build Status
Buildable 25
Build 25: arc lint + arc unit

Event Timeline

dereckson updated this revision to Diff 79.Nov 8 2015, 22:59
dereckson retitled this revision from to PhpFpmEnvironment: get → printConfig.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: Sandlayth.
Herald added a project: Nasqueron Docker deployment squad. · View Herald TranscriptNov 8 2015, 22:59
Sandlayth accepted this revision.Nov 9 2015, 21:29
Sandlayth edited edge metadata.
This revision is now accepted and ready to land.Nov 9 2015, 21:29
Closed by commit rDPHPFPMb32adb190b79: PhpFpmEnvironment: get → printConfig (authored by dereckson, committed by dereckson). · Explain WhyNov 9 2015, 23:18
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
files/
usr/
local/
lib/
php-fpm-env/
6 lines

Diff 79

View Options

files/usr/local/lib/php-fpm-env/PhpFpmEnvironment.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator