-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Make amphp/windows-registry a suggested dependency #63
Comments
That would make any app and library incompatible with Windows by default, so I'm against that. |
not incompatible by default as it's suggested & have to be documented but I understand your argument of course. |
The point is composer don't permit ATM to express requirements that vary depending on OS or php version, which would be great & make a more decent solution of course. |
Right, this is an issue Composer needs to solve first. I very much prefer a little bit of space in favor of compatible by default. |
As it's platform related and make vendor dir heavier for nothing when not running code on Windows.
The text was updated successfully, but these errors were encountered: