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.

Sign up for GitHub

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

Jump to bottom

Hotfix #279 #3

Closed
weierophinney opened this issue Dec 31, 2019 · 4 comments
Closed

Hotfix #279 #3

weierophinney opened this issue Dec 31, 2019 · 4 comments
Assignees
@GeeH
Labels
Bug Something isn't working
Milestone
3.5.0

Comments

@weierophinney
Copy link
Member

Fix issue #279

Originally posted by @thomasvargiu at zendframework/zend-servicemanager#280
@weierophinney
Copy link
Member Author

@thomasvargiu —

Run composer cs-fix, commit, and push, please. 😄

Originally posted by @weierophinney at zendframework/zend-servicemanager#280 (comment)

@weierophinney
Copy link
Member Author

@weierophinney fixed it

Originally posted by @thomasvargiu at zendframework/zend-servicemanager#280 (comment)

@weierophinney
Copy link
Member Author

@thomasvargiu It looks like a hotfix, so I think you should target master here.

Originally posted by @michalbundyra at zendframework/zend-servicemanager#280 (comment)

@weierophinney
Copy link
Member Author

@webimpress I rewrite it on master base

Originally posted by @thomasvargiu at zendframework/zend-servicemanager#280 (comment)

@weierophinney weierophinney mentioned this issue Dec 31, 2019
Open
@GeeH GeeH added this to the 3.5.0 milestone Jun 5, 2020
@GeeH GeeH added the Bug Something isn't working label Jun 5, 2020
@GeeH GeeH self-assigned this Jun 5, 2020
This was referenced Jun 5, 2020
Merged
Closed
Ocramius added a commit that referenced this issue Jun 12, 2020
@Ocramius
Merge pull request #43 from GeeH/geeh/#3-hotfix-from-old-repo
cc4c218 
#3 Ported 279 from ZF repository - fixes merging of duplicate delegator factory definitions
@GeeH GeeH closed this as completed Jun 12, 2020
boesing added a commit to boesing/laminas-servicemanager that referenced this issue Dec 2, 2020
@boesing
Merge pull request laminas#43 from GeeH/geeh/laminas#3-hotfix-from-ol…
f476f7e 
…d-repo
@MeetKamal MeetKamal mentioned this issue Apr 14, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GeeH GeeH

Labels
Bug Something isn't working
Projects
None yet
Milestone
3.5.0
Development

No branches or pull requests
2 participants
@weierophinney @GeeH