Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Conversation

@michalbundyra
Copy link
Member

Q A
QA yes

Description

Port of zendframework/zend-config#66

Fixes #1

@michalbundyra michalbundyra added the Bug Something isn't working label Apr 17, 2020
@samsonasik samsonasik merged commit a0f4958 into laminas:master Apr 18, 2020
samsonasik added a commit that referenced this pull request Apr 18, 2020
Forward port #7

* michalbundyra-hotfix/lowercase-config-key-sm:
  Updates copyright notes
  Set version constraint for replaced repository in composer.json
  Adds .github directory into .gitattributes
  Require specific repository_dispatch event type - docs-build
  Use lowercase `config` key to get configuration from container
@samsonasik
Copy link
Member

@michalbundyra Thank you, merged to master, forward port to develop.

@geerteltink
Copy link
Member

geerteltink commented Apr 18, 2020

Just wondering... isn't this a BC break for laminas?

AFAIK $array['config'] !== $array['Config']. This might break laminas components and projects based on this which expect the config key to be Config.

@samsonasik
Copy link
Member

@geerteltink
Copy link
Member

Ah cool, I didn't know that.

@michalbundyra michalbundyra deleted the hotfix/lowercase-config-key-sm branch April 18, 2020 07:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use lowercase config key to get configuration from container

3 participants