Skip to content
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

Minor cleanup #51

Merged
merged 20 commits into from
Aug 2, 2022
Merged

Minor cleanup #51

merged 20 commits into from
Aug 2, 2022

Conversation

whikloj
Copy link
Owner

@whikloj whikloj commented Aug 2, 2022

Switches to using / as file path separator internally for all OS'es.
Fixes to get Windows to pass tests.

@codecov
Copy link

codecov bot commented Aug 2, 2022

Codecov Report

Merging #51 (b7be6d8) into main (f2496ce) will increase coverage by 0.03%.
The diff coverage is 89.47%.

@@             Coverage Diff              @@
##               main      #51      +/-   ##
============================================
+ Coverage     88.61%   88.65%   +0.03%     
+ Complexity      491      485       -6     
============================================
  Files             7        7              
  Lines          1274     1260      -14     
============================================
- Hits           1129     1117      -12     
+ Misses          145      143       -2     
Impacted Files Coverage Δ
src/Bag.php 86.03% <81.25%> (+0.15%) ⬆️
src/BagUtils.php 92.39% <92.30%> (-0.47%) ⬇️
src/Commands/ValidateCommand.php 91.17% <100.00%> (ø)
src/Fetch.php 95.02% <100.00%> (-0.05%) ⬇️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@whikloj whikloj merged commit c6d0be5 into main Aug 2, 2022
@whikloj
Copy link
Owner Author

whikloj commented Aug 2, 2022

Thanks for the help @jonasraoni (🥇 ), once donatj/mock-webserver#42 is merged then I can try re-enabling the Fetch tests too.

@jonasraoni
Copy link
Contributor

Ok, I tested locally and it worked fine for me 👌

@whikloj whikloj deleted the minor-cleanup branch August 3, 2022 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants