Skip to content
This repository was archived by the owner on Feb 15, 2025. It is now read-only.

Fix issue with checking nullable return statements. #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Skorzhyk
Copy link

There is an issue with checking PHP docs for methods with multiple return statements like this:

/**
 * @return string|null
 */
public function maybeGetSomeString(): ?string {...}

The tool returns Warning for this, so it has been fixed in the PR.

@Skorzhyk Skorzhyk force-pushed the check-multiple-return-statements branch from 5c4c897 to fd783c9 Compare November 29, 2019 10:52
@Skorzhyk Skorzhyk force-pushed the check-multiple-return-statements branch 2 times, most recently from 4d9731b to 5b677a0 Compare November 29, 2019 12:05
@kaurov
Copy link

kaurov commented Sep 7, 2021

Who can resolve and merge this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants