Don't continue when raising a UnicodeDecodeError

[mattrobenolt]

Don't continue executing more code, by adding a continue to start the loop over. :)

[alexcouper]

The travis build failed for this. Could you rebase from develop? You should then get some green travis love.

[mattrobenolt]

@alexcouper Done.

[mattrobenolt]

All green. :)

[mattrobenolt]

Bump?

[diox]

Hey, sorry I had not seen this - it needs to be rebased again (can't merge it atm), and needs a test.

[diox]

Closing per lack of reply.

[mattrobenolt]

@diox This is still a bug fwiw, and the patch is super simple. So not sure why this was closed.

[diox]

It was closed because there had not been a reply in over a year, and it had no tests and wasn't applying to latest develop branch. I'll gladly merge it if those 2 issues are fixed!

[mattrobenolt]

I can happily apply to the develop branch, but not sure how to test this since I'm not intimate with the code base. The logic is trivial and this was obviously a typo when originally written since every other except statement has a continue which ends up resulting in a fatal NameError on template not existing.

We see this in our CI builds all the time.

[mattrobenolt]

@diox I just rebased and pushed to my branch, but the PR didn't update now since it's closed, but the commit is mattrobenolt/django-compressor@31454b0

[diox]

Ok, I can't re-open the PR, so can you submit a new PR with that please ?