Homec4science

Drive all Celerity operations from the new map

Authored by epriestley <git@epriestley.com> on Jan 1 2014, 03:04.

Description

Drive all Celerity operations from the new map

Summary:
Ref T4222.

  • Removes the old map and changes the CelerityResourceMap API to be entirely driven by the new map.
  • The new map is about 50% smaller and organized more sensibly.
  • This removes the /pkg/ URI component. All resources are now required to have unique names, so we can tell if a resource is a package or not by looking at the name.
  • Removes some junky old APIs.
  • Cleans up some other APIs.
  • Added some feedback for bin/celerity map.
  • CelerityResourceMap is still a singleton which is inextricably bound to the Phabricator map; this will change in the future.

Test Plan:

  • Reloaded pages.
  • Verified packaging works by looking at generated includes.
  • Forced minification on and verified it worked.
  • Forced no-timestamps on and verified it worked.
  • Rebuilt map.
  • Ran old script and verified error message.
  • Checked logs.

Reviewers: btrahan, hach-que

Reviewed By: hach-que

CC: chad, aran

Maniphest Tasks: T4222

Differential Revision: https://secure.phabricator.com/D7872

Details

Committed
epriestley <git@epriestley.com>Jan 1 2014, 03:04
Pushed
aubortJan 31 2017, 17:16
Parents
rPH60cb65bfbb2d: Make resolveResources() and packageResources() private on CelerityResourceMap
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH2c35532256ea: Drive all Celerity operations from the new map (authored by epriestley <git@epriestley.com>).Jan 1 2014, 03:04