-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tags for file-specific paths should normalise pathnames #146
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #146 +/- ##
=========================================
Coverage 97.89% 97.90%
- Complexity 849 850 +1
=========================================
Files 37 37
Lines 2517 2524 +7
=========================================
+ Hits 2464 2471 +7
Misses 53 53 ☔ View full report in Codecov by Sentry. |
2e9ab4e
to
715b43e
Compare
715b43e
to
3639ade
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @andrewnicols!
Thanks for working on this improvement. It's great to see Windows support too. Well done! I love the way you split commits (it always helps to better understanding the code) <3
I'm happy to approve this change and merge it :-)
No description provided.