You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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?
The text was updated successfully, but these errors were encountered:
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
IsFileMatch
andIsDirectoryMatch
methods, so it's really explicit?The text was updated successfully, but these errors were encountered: