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

feat(ini) support different settings for php.ini #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

scambra
Copy link
Contributor

@scambra scambra commented Feb 19, 2020

When php:version is a list, a dict with settings for different
versions can be added under ini key of cli and fpm:config keys

php:
  fpm:
    config:
      ini:
        settings:
          # global settings
        '5.6':
          # settings for 5.6 only
  cli:
    ini:
      settings:
        # global settings
      '5.6':
        # settings for 5.6 only

Related to #138, grandmotivator comment

@myii
Copy link
Member

myii commented Mar 11, 2020

@scambra Sorry for the delay, we need to get some eyes on this from those who use the formula. How about @grandmotivator? Could you try this out?

When php:version is a list, a dict with settings for different
versions can be added under ini key of cli and fpm:config keys

php:
  fpm:
    config:
      ini:
        settings:
          # global settings
        '5.6':
          # settings for 5.6 only
  cli:
    ini:
      settings:
        # global settings
      '5.6':
        # settings for 5.6 only
@myii
Copy link
Member

myii commented Jun 16, 2020

@sticky-note Would you be able to review this as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants