search

j0hnsmith / django-pipeline-browserify

MIT License
37 stars 18 forks source link

bug fix #11

Closed lebeier closed 8 years ago

lebeier commented 8 years ago

changed getattr to get for pipeline_settings added "browserified" back into the output_extension

lebeier commented 8 years ago

I just realized that I accidentally removed the change while rebasing 15 days ago.

zanderle commented 8 years ago

Will this be merged soon? The compiler doesn't work without this fix.

j0hnsmith commented 8 years ago

sorry for the delay

j0hnsmith commented 8 years ago

@zanderle can you confirm that it works and I'll push a new version to pypi

zanderle commented 8 years ago

I can confirm it works, but it would probably be a good idea to add a simple Requires django-pipeline >= 1.6 in README, since it won't work otherwise.

j0hnsmith commented 8 years ago

@zanderle I made django-pipeline>=1.6.0 a dependency and pushed to pypi. Thanks for your help.