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

Fix returned value of Laminas\Config\Processor\Queue#process() to comply with interface declaration #13

Merged
merged 1 commit into from
Sep 8, 2021

Conversation

7-zete-7
Copy link
Contributor

Laminas\Config\Processor\Queue implements Laminas\Config\Processor\ProcessorInterface, but lying about return value on process() method.

Signed-off-by: GitHub <noreply@github.com>
This was referenced Jul 14, 2021
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - will merge up from here though, no need for multiple PRs to multiple branches :)

@Ocramius Ocramius added this to the 3.4.1 milestone Sep 8, 2021
@Ocramius Ocramius added the Bug Something isn't working label Sep 8, 2021
@Ocramius Ocramius self-assigned this Sep 8, 2021
@Ocramius Ocramius merged commit a373b2e into laminas:3.4.x Sep 8, 2021
@Ocramius Ocramius changed the title Fix of interface violations Fix returned value of Laminas\Config\Processor\Queue#process() to comply with interface declaration Sep 8, 2021
@7-zete-7 7-zete-7 deleted the 3.4.x branch September 16, 2021 16:30
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.

2 participants