Skip to content
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

zf1s/zf1: require php extensions in dev only #136

Merged
merged 1 commit into from
Nov 8, 2022

Commits on Nov 8, 2022

  1. require php extensions in dev only

    It is probably too restrictive to have them listed in `require` section in the root `composer.json` file.
    
    Due to the nature of zf1, you may never need some of the extensions, as long as you do not use components which require them. But when you start using them, those components themselves should throw errors that required extensions are not installed.
    
    Moved to `require-dev` to ensure they are installed when running tests on dev/CI.
    
    I've considered adding them to `suggest` section but imo composer might spam too much with irrelevant information, when installing `zf1s/zf1` package.
    falkenhawk committed Nov 8, 2022
    Configuration menu
    Copy the full SHA
    8a1ab61 View commit details
    Browse the repository at this point in the history