Page MenuHomePhabricator
Differential D17422

Add a lipsum generator for Badges
ClosedPublic
Actions

Authored by epriestley on Feb 27 2017, 1:18 PM.
Tags
None
Referenced Files
F13186253: D17422.diff
Sat, May 11, 3:31 AM
Unknown Object (File)
Tue, May 7, 6:32 AM
Unknown Object (File)
Fri, May 3, 4:01 AM
Unknown Object (File)
Mon, Apr 29, 4:15 PM
Unknown Object (File)
Fri, Apr 26, 11:27 PM
Unknown Object (File)
Wed, Apr 24, 11:58 PM
Unknown Object (File)
Apr 11 2024, 8:16 AM
Unknown Object (File)
Apr 9 2024, 10:14 AM
View All Files
Subscribers
cspeckmim

Details

Reviewers
chad
Maniphest Tasks
T12270: Unbeta Badges
T12319: Investigate an Owners-related performance issue on revision detail pages
Commits
rP5cb4c76bef6b: Add a lipsum generator for Badges
Summary

Ref T12319. Ref T12270. Allow badges to be generated with bin/lipsum. These aren't hugely sophisticated but I'm not sure about the fate of T9010 yet or what's happening with the quality levels, and didn't want to make those changes more difficult.

Test Plan
  • Used bin/lipsum generate badges --force --quickly to generate badges.
  • Made some coffee and came back to 20K badges.

Screen Shot 2017-02-27 at 4.59.13 AM.png (1×2 px, 305 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 27 2017, 1:18 PM
Harbormaster completed remote builds in B15792: Diff 41891.Feb 27 2017, 1:19 PM
cspeckmim added a subscriber: cspeckmim.Feb 27 2017, 2:00 PM

diablo.gif (480×640 px, 185 KB)

epriestley added a comment.Feb 27 2017, 2:56 PM

I was using this source if anyone wants to send the affixes so we can generate Obsidian Ring of the Zodiac:

http://diablo.gamepedia.com/Affixes_(Diablo_I)

chad accepted this revision.Feb 27 2017, 3:43 PM
This revision is now accepted and ready to land.Feb 27 2017, 3:43 PM
epriestley mentioned this in T12270: Unbeta Badges.Feb 27 2017, 4:10 PM
Closed by commit rP5cb4c76bef6b: Add a lipsum generator for Badges (authored by epriestley, committed by epriestley). · Explain WhyFeb 27 2017, 5:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 41897

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/badges/lipsum/PhabricatorBadgesBadgeTestDataGenerator.php

Loading...
View Options

src/applications/badges/lipsum/PhabricatorBadgesLootContextFreeGrammar.php

Loading...
Phabricator · Built by Phacility