Page MenuHomePhabricator
Differential D18147

Add a cache purger for builtin files
ClosedPublic
Actions

Authored by epriestley on Jun 22 2017, 5:56 PM.

Details

Reviewers
chad
Maniphest Tasks
T12859: Some means of updating builtin images without a manual purge
Commits
rP224c4692ee0e: Add a cache purger for builtin files
Summary

Fixes T12859.

Test Plan
  • Loaded Diffusion builtin icons before recent updates, saw cached builtins.
  • Ran bin/cache purge --caches builtin-file.
  • Reloaded, saw new files.

Diff Detail

Repository
rP Phabricator
Branch
purger2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17550
Build 23550: Run Core Tests
Build 23549: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 22 2017, 5:56 PM
Harbormaster completed remote builds in B17550: Diff 43662.Jun 22 2017, 5:57 PM
chad accepted this revision.Jun 22 2017, 6:07 PM
This revision is now accepted and ready to land.Jun 22 2017, 6:07 PM
Closed by commit rP224c4692ee0e: Add a cache purger for builtin files (authored by epriestley, committed by epriestley). · Explain WhyJun 22 2017, 6:14 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Jun 22 2017, 6:16 PM

cool nice

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
cache/
management/
4 lines
purger/
22 lines
files/
query/
18 lines

Diff 43662

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/cache/management/PhabricatorCacheManagementPurgeWorkflow.php

Loading...
View Options

src/applications/cache/purger/PhabricatorBuiltinFileCachePurger.php

Loading...
View Options

src/applications/files/query/PhabricatorFileQuery.php

Loading...
Phabricator · Built by Phacility