lunarmodules / luacov

LuaCov is a simple coverage analyzer for Lua code.
http://lunarmodules.github.io/luacov/
MIT License
300 stars 68 forks source link

Update file exclusion/inclusion lists #4

Closed Tieske closed 10 years ago

Tieske commented 10 years ago

Updated

hishamhm commented 10 years ago

removed ambiguity with patterns where '.' was used as path separator (now standardized on '/')

Won't this introduce an incompatibility and break things for users who are relying on '.'?

Tieske commented 10 years ago

Theoretically; yes.

But as nobody ever complained about it not working... it won't.

hishamhm commented 10 years ago

oh :)

Tieske commented 10 years ago

come to think of it; no. it does not break compatibility. The old patterns for matching a file were more permissive than the new ones. So the old ones a user might have put in a config file will still work. The old ones contain '.' for path separators, the new ones '/'. So the new ones match exactly, while the old ones match based on the magic patern character '.'.

Tieske commented 10 years ago

unless of course someone actually made the effort to escape the pattern by including '%.'....