[GH-ISSUE #227] Add argument and return type typehints #563

Open
opened 2026-03-14 12:13:10 +03:00 by kerem · 3 comments
Owner

Originally created by @rimas-kudelis on GitHub (May 27, 2025).
Original GitHub issue: https://github.com/antonioribeiro/google2fa/issues/227

It's a bit weird to see the code using PHP attributes, but not documenting argument or return types, nor using strict types.

Would you consider adding these things and releasing a new major version?

Originally created by @rimas-kudelis on GitHub (May 27, 2025). Original GitHub issue: https://github.com/antonioribeiro/google2fa/issues/227 It's a bit weird to see the code using PHP attributes, but not documenting argument or return types, nor using strict types. Would you consider adding these things and releasing a new major version?
Author
Owner

@antonioribeiro commented on GitHub (May 27, 2025):

Have you seen the work being done here? https://github.com/antonioribeiro/google2fa/commits/9.x/

<!-- gh-comment-id:2913908171 --> @antonioribeiro commented on GitHub (May 27, 2025): Have you seen the work being done here? https://github.com/antonioribeiro/google2fa/commits/9.x/
Author
Owner

@rimas-kudelis commented on GitHub (May 28, 2025):

Not really. Your README says version 8 is still in beta, and this is version 9. 😆

If you've done that already, I'm glad and looking forward to a release!

<!-- gh-comment-id:2915191499 --> @rimas-kudelis commented on GitHub (May 28, 2025): Not really. Your README says version 8 is still in beta, and this is version 9. 😆 If you've done that already, I'm glad and looking forward to a release!
Author
Owner

@antonioribeiro commented on GitHub (May 28, 2025):

A lot of work to be done there still, but if you can help I can probably merge your PRs 😉

<!-- gh-comment-id:2917793162 --> @antonioribeiro commented on GitHub (May 28, 2025): A lot of work to be done there still, but if you can help I can probably merge your PRs 😉
Sign in to join this conversation.
No labels
bug
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/google2fa#563
No description provided.