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
Unknown Object (File)
Fri, Apr 19, 7:50 PM
Unknown Object (File)
Tue, Mar 26, 3:47 PM
Unknown Object (File)
Mar 23 2024, 9:42 PM
Unknown Object (File)
Mar 12 2024, 4:25 PM
Unknown Object (File)
Feb 3 2024, 7:08 PM
Unknown Object (File)
Jan 21 2024, 9:03 PM
Unknown Object (File)
Jan 21 2024, 7:35 PM
Unknown Object (File)
Jan 3 2024, 8:06 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)