Differential D9632

Fix `PhutilReadableSerializer` to quote strings.
ClosedPublic
Actions

Authored by joshuaspence on Jun 19 2014, 8:35 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUd8935319d556: Fix `PhutilReadableSerializer` to quote strings.

Summary

See D9622#inline-37279 for explanation.

Test Plan

Modified and ran unit tests.

Diff Detail

Repository

rPHU libphutil

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 1245
Build 1245: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence updated this revision to Diff 23100.Jun 19 2014, 8:35 PM

joshuaspence retitled this revision from to Fix `PhutilReadableSerializer` to quote strings..

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 19 2014, 8:35 PM

Herald added subscribers: Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1245: Diff 23100.Jun 19 2014, 8:36 PM

epriestley accepted this revision.Jun 19 2014, 8:44 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 19 2014, 8:44 PM

Closed by commit rPHUd8935319d556 (authored by @joshuaspence).

Revision Contents
Changeset List

Path

Size

src/

readableserializer/

PhutilReadableSerializer.php

9 lines

__tests__/

PhutilReadableSerializerTestCase.php

2 lines

Diff 23100

View Options

src/readableserializer/PhutilReadableSerializer.php

View Options

src/readableserializer/tests/PhutilReadableSerializerTestCase.php

Fix `PhutilReadableSerializer` to quote strings.ClosedPublicActions