-
Notifications
You must be signed in to change notification settings - Fork 226
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
Wrong types #305
Comments
🙃 well, crud. Sorry about that. If you'd like to PR the fixes to the docs, please feel free to do so! I don't have an ETA on when these will get fixed up, but they will eventually get fixed. |
Going to turn this first issue on types into a mega-thread of issues similar issues opened. |
related PRs - #315 |
for internal tracking, ticket 1401 |
Lumping in #326 with this as well, in that this PR highlights that returns and throws need cleaning up. |
General information
Issue description
I try to use v6.5.0 instead of our actual v6.4.1, but PhpStan found a lot of type errors.
$addressId
accepts null$signature
accepts and check null also$params
is array but expectsOptional
, what is not defined.sampleNotification
returns not definedWebhook
but actually returns arrayCommit e35c2a2 is full types mistakes like this. Is it possible to fix this?
The text was updated successfully, but these errors were encountered: