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
The blogpost indicated that our setup wouldn't be affected by this but the tool was handy to verify programmatically. Thank you!
I'd suggest a minor code change (L11) which will cover the case where '*' might not be present in one of the patterns
Recommendation: if "*" in pattern and pattern.index("*") < pattern.index("/"):
instead of if pattern.index("*") > 0 and pattern.index("*") < pattern.index("/"):
Happy to open a PR if you prefer that.
The text was updated successfully, but these errors were encountered:
The blogpost indicated that our setup wouldn't be affected by this but the tool was handy to verify programmatically. Thank you!
I'd suggest a minor code change (L11) which will cover the case where '*' might not be present in one of the patterns
Recommendation:
if "*" in pattern and pattern.index("*") < pattern.index("/"):
instead of
if pattern.index("*") > 0 and pattern.index("*") < pattern.index("/"):
Happy to open a PR if you prefer that.
The text was updated successfully, but these errors were encountered: