Currently we have public bool IsMatch(string path, bool pathIsDirectory), but it does cause a bit of "boolean blindness" unless you label the parameter... for example, without looking at the source, what does the second parameter mean in this case?
rule.IsMatch(directory.FullName, true);
Might be nicer to either replace it with an enum, or make that method internal (or even private) and add IsFileMatch and IsDirectoryMatch methods, so it's really explicit?
Currently we have
public bool IsMatch(string path, bool pathIsDirectory), but it does cause a bit of "boolean blindness" unless you label the parameter... for example, without looking at the source, what does the second parameter mean in this case?Might be nicer to either replace it with an enum, or make that method internal (or even private) and add
IsFileMatchandIsDirectoryMatchmethods, so it's really explicit?