Update to django-pipeline 1.6

Review Request #7953 — Created Feb. 9, 2016 and submitted

Information

Review Board
release-2.6.x
372193b...

Reviewers

This change makes the changes necessary in Review Board to use django-pipeline
1.6.0. In particular, we need to switch to the new ExtensionFinderStorage,
update the settings for the new PIPELINE attribute, and modify our templates
to use the new templatetags.

  • Went through the UI and verified CSS, JS, and images.
  • Checked to see that an extension still worked in dev-mode.
  • Ran unit tests.
  • Built a package and saw media collect correctly.
  • Built an extension package and saw that everything built correctly.
Description From Last Updated

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

Col: 1 E402 module level import not at top of file

reviewbotreviewbot

So this will be used during runtime only? Or am I misunderstanding? If so, we need STATIC_ROOT to consider subdir …

chipx86chipx86

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

Col: 1 E402 module level import not at top of file

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/settings.py
        reviewboard/attachments/mimetypes.py
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/js/tests.html
        reviewboard/templates/admin/login.html
        reviewboard/templates/base.html
        reviewboard/templates/admin/notifications/webhooktarget/change_form.html
        reviewboard/templates/admin/scmtools/repository/change_form.html
        reviewboard/templates/reviews/ui/default.html
        reviewboard/templates/admin/dashboard.html
        reviewboard/templates/admin/base_site.html
        reviewboard/templates/admin/security.html
        reviewboard/templates/reviews/new_review_request.html
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/reviews/reviewable_base.html
        reviewboard/templates/admin/diffviewer/filediff/change_form.html
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/settings.py
        reviewboard/attachments/mimetypes.py
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/js/tests.html
        reviewboard/templates/admin/login.html
        reviewboard/templates/base.html
        reviewboard/templates/admin/notifications/webhooktarget/change_form.html
        reviewboard/templates/admin/scmtools/repository/change_form.html
        reviewboard/templates/reviews/ui/default.html
        reviewboard/templates/admin/dashboard.html
        reviewboard/templates/admin/base_site.html
        reviewboard/templates/admin/security.html
        reviewboard/templates/reviews/new_review_request.html
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/reviews/reviewable_base.html
        reviewboard/templates/admin/diffviewer/filediff/change_form.html
    
    
  2. reviewboard/settings.py (Diff revision 1)
     
     
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 1)
     
     
     'from settings_local import *' used; unable to detect undefined names
    
  4. reviewboard/settings.py (Diff revision 1)
     
     
    Col: 1
     E402 module level import not at top of file
    
  5. 
      
brennie
  1. Ship It!
  2. 
      
chipx86
  1. 
      
  2. reviewboard/settings.py (Diff revision 1)
     
     
     
     
     
     
     
     
     
     
     
     

    So this will be used during runtime only? Or am I misunderstanding?

    If so, we need STATIC_ROOT to consider subdir installs.

    Also, we're duplicating a lot here from contrib/internal/conf/settings_local.py, and also the Djblets packaging stuff. Wonder if we can centralize more of it.

    1. Hmm, so yeah, that's an issue for this particular patch. However, with https://reviews.reviewboard.org/r/7954/ we'll be able to simplify all of this because compilation will happen on the backend.

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/settings.py
        reviewboard/attachments/mimetypes.py
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/js/tests.html
        reviewboard/templates/admin/login.html
        reviewboard/templates/base.html
        reviewboard/templates/admin/notifications/webhooktarget/change_form.html
        reviewboard/templates/admin/scmtools/repository/change_form.html
        reviewboard/templates/reviews/ui/default.html
        reviewboard/templates/admin/dashboard.html
        reviewboard/templates/admin/base_site.html
        reviewboard/templates/admin/security.html
        reviewboard/templates/reviews/new_review_request.html
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/reviews/reviewable_base.html
        reviewboard/templates/admin/diffviewer/filediff/change_form.html
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/settings.py
        reviewboard/attachments/mimetypes.py
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/js/tests.html
        reviewboard/templates/admin/login.html
        reviewboard/templates/base.html
        reviewboard/templates/admin/notifications/webhooktarget/change_form.html
        reviewboard/templates/admin/scmtools/repository/change_form.html
        reviewboard/templates/reviews/ui/default.html
        reviewboard/templates/admin/dashboard.html
        reviewboard/templates/admin/base_site.html
        reviewboard/templates/admin/security.html
        reviewboard/templates/reviews/new_review_request.html
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/reviews/reviewable_base.html
        reviewboard/templates/admin/diffviewer/filediff/change_form.html
    
    
  2. reviewboard/settings.py (Diff revision 2)
     
     
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 2)
     
     
     'from settings_local import *' used; unable to detect undefined names
    
  4. reviewboard/settings.py (Diff revision 2)
     
     
    Col: 1
     E402 module level import not at top of file
    
  5. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.6.x (93a591f)
Loading...