PhpFpmEnvironment: get → printConfig
ClosedPublic
Actions

Authored by dereckson on Nov 8 2015, 22:59.

Details

Reviewers

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

Path

Size

files/

usr/

local/

lib/

php-fpm-env/

PhpFpmEnvironment.php

6 lines

Diff 79

View Options

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

PhpFpmEnvironment: get → printConfigClosedPublicActions