fix: removes trailing /* from --ignore-glob

[wllfaria]

This PR removes trailing / or /* and should fix #865.

Where globs ending with those patterns would not match any directory, as the default listing doesn't include an ending /.

As mentioned here, the pattern generated by a glob that ends either in / or /*, such as node_modules/* or node_modules/* would not match as the default listing of directories don't include those characters.

Description

I added a line at that goes as follows:

let input = input.trim_end_matches("/*").trim_end_matches('/');

Removing both patterns when they exists at the end of the glob. Here is a screenshot of it working:

How Has This Been Tested?

Should I write new unit tests for this specific case?

[massivebird]

These changes do not appear to solve the linked issue. The issue's author is looking for the ability to ignore files inside some subdirectory without ignoring the subdirectory itself. This behavior appears to be missing from these changes.

[wllfaria]

@massivebird you're absolutely right, I'll consider how to implement this in a good way

[AliSananS]

@massivebird You are correct, knowing if node_modules directory exist is useful but not what's inside this. 😂

[wllfaria]

Ill add some comments on the issue, about explorations I did to fix this problem. Its not as straightforward as it may seem