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 laminas-zendframework-bridge. #13

Closed
wants to merge 1 commit into from

Conversation

Danack
Copy link

@Danack Danack commented Apr 30, 2021

Projects that use laminas-escaper that are not using it as part of a legacy zendframework application don't need the BC autoloading bridge.

See - #11 (comment)

Q A
Documentation no
Bugfix no
BC Break yes
New Feature no
RFC no
QA no

Description

  • Are you fixing a BC Break?

yes.

  • How do you reproduce it?

Composer update

  • What was the previous behavior?

Install zendbridge-framework.

  • What is the current behavior?

Don't install zendbridge-framework.

Projects that use laminas-escaper that are not using it as part of a legacy zendframework application don't need the BC autoloading bridge.

Signed-off-by: Danack <Danack@basereality.com>
@weierophinney
Copy link
Member

@Ocramius already resolved with #27 anyways, so I'm closing this and removing it from the 2.9.0 milestone.

@weierophinney weierophinney removed this from the 2.9.0 milestone Sep 2, 2021
@Ocramius
Copy link
Member

Ocramius commented Sep 2, 2021

@weierophinney I added the milestone so it shows up in release notes

@Ocramius Ocramius added this to the 2.9.0 milestone Sep 2, 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.

4 participants