Only compile files with .html extension. #39

Merged
merged 2 commits into from Aug 16, 2012

Projects

None yet

2 participants

@runejuhl
Contributor

Fixes the issue at ChicagoBoss/ChicagoBoss#140 , wrongly attributed to Chicago Boss.

@evanmiller
Contributor

Some people use other extensions (.dtl, .txt, etc.). I think the patch should just exclude files that end with a tilde rather than require the .html extension.

@runejuhl
Contributor

ErlyDTL (or is it Chicago Boss?) only accepts views with the ".html" ending, which is why I chose this.

I'm not sure excluding files with a ending tilde is a good way to go, as backup files might have a trillion different endings, e.g. .bak or .old. I'll be glad to change it, but I'd prefer something in the way of (html)|(djhtml)|(dtl).

@evanmiller
Contributor

ErlyDTL accepts templates with any file extension. I think just excluding tildes is the way to go. There might be other backup file endings but I would prefer to deal with them on a case-by-case basis.

@evanmiller
Contributor

Thanks!

@evanmiller evanmiller merged commit 8552440 into erlydtl:master Aug 16, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment