Andi Rückauer
Andi Rückauer
I would like to first get an overview of all required changes. Can we open a separate branch for that? There we could build a prototype (for makefile, travis setup...
> I like the idea of having an `example` directory Yes, I believe a separate directory for the example code has the benefits of a clear separation and allows easier...
Yes, that's possible. Although it introduces two issues: It makes it impossible to 1. differentiate between files containing a class and files containing function(s) and 2. to define a class...
Thanks for the follow-up. It's exciting to see all the changes coming to the repo. 👏🏻 Wherever possible I would encourage the usage of best practices for a particular language...
That's a good idea, it's common practice nowadays and very helpful to have a tool taking care of it. I think the two most commonly used linters are [PHP-CS-Fixer](https://github.com/FriendsOfPHP/PHP-CS-Fixer) and...
@neenjaw Awesome! Gobally I exclude the `PSR1.Methods.CamelCapsMethodName` rule for test classes. Descriptive test method names can be rather long and I write them in snake_case for improved readability. Other than...
PSR-12 is already conigured (see [phpcs-php.xml](https://github.com/exercism/php/blob/main/phpcs-php.xml)). Since the PSR-12 is providing "only" very basic rules. This discussion is about having a bit more precise standard.
This error is reproducable on Windows 10 for all versions starting from `1.0.0` and caused by Windows ignoring double quotes in passing a string argument in the composer `post-create-project-cmd` script....
I wanted to provide a PR, updating the documentation. However, `ClassMapAutoloader` documentation suggests the usage of `classmap_generator.php`. Is there a valid use case for `ClassMapAutoloader`, without the generator? How would...
I was not able to reproduce this error. It seems, it is fixed in the current version. Successfully tested on Windows 10 Pro with php-7.1.26-Win32-VC14-x64, php-7.2.15-Win32-VC15-x64 and php-7.3.2-Win32-VC15-x64. Used quickstart...