Skip to content
  • epriestley's avatar
    Drive all Celerity operations from the new map · 2c355322
    epriestley authored
    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
    2c355322