Skip to content

[CLEANUP] Rector: Add union return type #604

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

Closed

Conversation

ziegenberg
Copy link
Contributor

@@ -26,12 +26,10 @@ public function __construct(array $aColor, $iLineNo = 0)
* @param ParserState $oParserState
* @param bool $bIgnoreCase
*
* @return Color|CSSFunction
Copy link
Collaborator

Choose a reason for hiding this comment

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

For those cases with a real union type, we'll need to keep the PHPDoc.

@oliverklee
Copy link
Collaborator

I have created #608 to add native return types to parse(). Maybe we can get that in first.

@ziegenberg ziegenberg closed this Jun 24, 2024
@ziegenberg ziegenberg force-pushed the add-type-coverage-level-26 branch from ca9b3b8 to 1908a05 Compare June 24, 2024 12:29
@ziegenberg ziegenberg deleted the add-type-coverage-level-26 branch June 24, 2024 12:30
@ziegenberg
Copy link
Contributor Author

I rebased and it closed on itself.

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

Successfully merging this pull request may close these issues.

2 participants