This repository was archived by the owner on Jan 31, 2020. It is now read-only.
Fix typos in DirWriteable check + add unit test based on vfsStream #65
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi!
First of all i want to say "thank you" for these components! We use them in production together with the Liip Monitor bundle and its very very helpful. So: Thanks for your work!
I recently spotted a small typo in the "DirWritable" check. There was an doubled "a" in the message "The path is a writable directory." Additionally i added some missing dots. :)
I tried to fix it with the existing unit-tests but unfortunately they do not work for me mainly because i'm on a Windows system (chmod stuff simply doesnt work on win machines). So i decided to add a new testsuite especially for the DirWriteable check implementing checks using a virtual filesystem based on vfsStream.
Actually i dont know if this pr breaks some of the existing tests. There is no need to add my tests when you fix the typo by yourself. :)
Thanks again.
Regards
Kai