storborg / pyramid_frontend

Theme handling, image filtering, and asset optimization for Pyramid
Other
0 stars 2 forks source link

Refactor asset type specifiers to use a class system, instead of tuples #21

Closed storborg closed 10 years ago

storborg commented 10 years ago

For example, instead of an asset specification that looks like:

assets = {
    'main-less': ('/_eagle/css/main.less', 'less'),
    'main-js': ('/_eagle/js/theme.js', 'requirejs'),
}

go with something like:

assets = {
    'main-less': LessAsset('/_eagle/css/main.less'),
    'main-js': RequireJSAsset('/_eagle/js/theme.js'),
}

This has a couple of advantages:

storborg commented 10 years ago

Done in 8a8668855b99d9058d044b952115e630b24de6c0