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

Remove mentioning of deprecated mezzio-config-manager #39

Merged
merged 1 commit into from
Sep 29, 2021
Merged

Remove mentioning of deprecated mezzio-config-manager #39

merged 1 commit into from
Sep 29, 2021

Conversation

arueckauer
Copy link
Member

Q A
Documentation yes/no

Description

The original package mtymek/expressive-config-manager is deprecated and was merged to zendframework/zend-config-aggregator and finally moved to laminas/laminas-config-aggregator.

The package (mezzio-config-manager) was falsely renamed during Rewriting as Laminas Project package and since a dead link.

Since essentially the same package is referenced twice, the dead link is removed instead of replaced.

Fixes #37

Signed-off-by: Andi Rückauer <arueckauer@gmail.com>
@boesing
Copy link
Member

boesing commented Sep 29, 2021

Thanks, @arueckauer!

@boesing boesing merged commit d1ff0b5 into laminas:2.3.x Sep 29, 2021
@boesing
Copy link
Member

boesing commented Sep 29, 2021

Oh damn, this one targeted an old branch. Not sure if its worth releasing 3 patches to make it to the latest release branch.

any reason why this was pointing to such an old branch?

@Ocramius
Copy link
Member

@boesing tbh, can just do the releases - a bit noisy, but otherwise harmless

@boesing
Copy link
Member

boesing commented Sep 29, 2021

@Ocramius I am on mobile only so I cannot fix changelog conflicts 😩

@Ocramius
Copy link
Member

Same 🤣

@arueckauer arueckauer deleted the documentation/remove-deprecated-expressive-config-manager branch September 30, 2021 03:33
@boesing boesing added this to the 2.3.3 milestone Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants