Page MenuHomePhabricator

Fix Lipsum generators for Differential Revisions and Pastes
ClosedPublic

Authored by kaendfinger2 on Jul 22 2018, 5:52 PM.
Tags
None
Referenced Files
F19153811: D19534.diff
Dec 11 2025, 12:13 PM
F19153775: D19534.diff
Dec 11 2025, 12:06 PM
F19050373: D19534.id.diff
Nov 27 2025, 7:56 PM
F18831926: D19534.id.diff
Oct 25 2025, 5:00 PM
F18762712: D19534.id46705.diff
Oct 6 2025, 8:19 PM
F18744740: D19534.diff
Oct 3 2025, 4:01 AM
F18670680: D19534.diff
Sep 25 2025, 1:06 AM
F18622802: D19534.id46700.diff
Sep 15 2025, 1:02 PM
Subscribers

Details

Summary

When generating test data to solve a bug I have encountered, I noticed Lipsum was not working correctly for Differential Revisions and Pastes.

It seemed like they weren't updated after some refactoring. This fixes that by updating them.

Test Plan

Run Lipsum for all objects, and note that it has much less failure.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This looks like it has some 4-space indents. In this project, for consistency, prefer 2-space indents.

I added you to Community (granting you sweeping janitorial powers on this install) and Blessed Committers, allowing you to land this yourself. That project's description has some instructions for pushing. Yell if you run into trouble.

This revision is now accepted and ready to land.Jul 23 2018, 7:53 PM
This revision was automatically updated to reflect the committed changes.

@epriestley Thank you very much. I've landed this patch. I've got a few more things I've discovered I'll be submitting for review as well. I'll make sure to use two spaces instead of four (I think this is because PhpStorm sets a 'continuation indent' to four and not two)