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.

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

fix psalm errors #11

Merged
merged 3 commits into from
Mar 24, 2021
Merged

fix psalm errors #11

merged 3 commits into from
Mar 24, 2021

Conversation

Lctrs
Copy link
Contributor

@Lctrs Lctrs commented Mar 24, 2021

No description provided.

You have to specify the type of the value
phpdoc say it should return a Option<U>, but $this is of type Option<T>, so psalm detect this as a type mismatch
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1f37ee0 on Lctrs:fix/psalm-errors into e060cfe on prewk:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1f37ee0 on Lctrs:fix/psalm-errors into e060cfe on prewk:master.

@Lctrs
Copy link
Contributor Author

Lctrs commented Mar 24, 2021

The two remaining errors will be fixed once $pass phpdoc type is fixed on Result side.

@prewk prewk merged commit fa938d7 into prewk:master Mar 24, 2021
@Lctrs Lctrs deleted the fix/psalm-errors branch March 24, 2021 10:31
@Lctrs
Copy link
Contributor Author

Lctrs commented Mar 24, 2021

@prewk You can tag the new major now :)

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

Successfully merging this pull request may close these issues.

3 participants