Bootstrap php package #30
Conversation
| }, | ||
| "autoload": { | ||
| "psr-4": { | ||
| "Dzcodeio\\Kuliya\\": "src/" |
There was a problem hiding this comment.
I don't understand well php and composer, but if it's possible, can u make all files/dirs lowercased ( either python-cased or snake_cased ) to avoid across OSs file-system issues
There was a problem hiding this comment.
Most popular php projects/frameworks use PascalCase for file/dir names so i dont think that would cause any issues but i can use snake_case starting from the next PR. As for namespaces, Class/Method names and class file names i chose to respect PHP standard recommendations, specificaly PSR-4 for class autoloading and PSR-12 for code style which suggests adopting PascalCase for namespaces/classes and class file names and camelCase for methods.
There was a problem hiding this comment.
In that case then, it's better to keep it consistent across the php folder, so let it be PascalCased 👌🏽.
Just note that later on, changing letter cases will cause problems on windows.
There was a problem hiding this comment.
Ahh i see! thanks for the hint, i'll investigate on it then and use the best strategy possible.
| }, | ||
| "autoload-dev": { | ||
| "psr-4": { | ||
| "Dzcodeio\\Kulyia\\Test\\": "tests/" |
Description
Bootstrap package by adding unit tests support and enabling class autoloading.
See also #10
Type of change