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.

Sign up for GitHub

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

Jump to bottom

IMAP resources have been converted to objects #6404

Closed
Tracked by #6395
weirdan opened this issue Sep 4, 2021 · 1 comment · Fixed by #6567
Closed
Tracked by #6395

IMAP resources have been converted to objects #6404

weirdan opened this issue Sep 4, 2021 · 1 comment · Fixed by #6567
Assignees
@tomasnorre
Labels
easy problems Issues that can be fixed without background knowledge of Psalm good first issue Help wanted internal stubs/callmap
Milestone
PHP 8.1

Comments

@weirdan
Copy link
Collaborator

weirdan commented Sep 4, 2021 •
edited
Loading

The IMAP functions now accept and return, respectively, IMAP\Connection objects instead of resources

Guide to editing callmaps: Editing callmaps.
Example of conversion: #6421
@weirdan weirdan mentioned this issue Sep 4, 2021
Closed
25 tasks
@weirdan weirdan added this to the PHP 8.1 milestone Sep 4, 2021
@weirdan weirdan added easy problems Issues that can be fixed without background knowledge of Psalm good first issue Help wanted internal stubs/callmap labels Sep 4, 2021
@tomasnorre
Copy link
Contributor

Could you please assign me to this one too? Then I'll give this a spin too.

@tomasnorre tomasnorre mentioned this issue Oct 2, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomasnorre tomasnorre

Labels
easy problems Issues that can be fixed without background knowledge of Psalm good first issue Help wanted internal stubs/callmap
Projects
None yet
Milestone
PHP 8.1
Development

Successfully merging a pull request may close this issue.

Convert imap functions to use object instead of a resource tomasnorre/psalm
2 participants
@weirdan @tomasnorre