Skip to content

Conversation

@Fan2Shrek
Copy link
Contributor

Using Docker in WSL does not seem to be correctly identified.

I tried to add Docker support, but I'm not sure if it's the best solution :)

Copy link
Member

@pyrech pyrech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Could you fix PHPStan issue, and add a note in the changelog and a test for the new method? Thanks

@Fan2Shrek
Copy link
Contributor Author

Done, but not sure if the test is relevant though.

@pyrech
Copy link
Member

pyrech commented Oct 2, 2024

Sorry for the delay, I forgot to-rereview your PR.

I agree It's a bit hard to make really useful tests on this library 😅 We may try to improve them in the future.

Let's merge your PR. Thanks for your work ✨

@pyrech pyrech merged commit 7062946 into jolicode:main Oct 2, 2024
3 checks passed
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