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
Skorzhyk wants to merge 1 commit intodancryer:masterfrom
Skorzhyk:check-multiple-return-statements
Open

Fix issue with checking nullable return statements.#38
Skorzhyk wants to merge 1 commit intodancryer:masterfrom
Skorzhyk:check-multiple-return-statements

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

Comments