Skip to content

Add type hints for mappers - #37511

Merged
provokateurin merged 1 commit into
masterfrom
feature/type-mappers
Apr 7, 2023
Merged

Add type hints for mappers#37511
provokateurin merged 1 commit into
masterfrom
feature/type-mappers

Conversation

@provokateurin

Copy link
Copy Markdown
Member

Summary

Split out from #37390

Checklist

Comment threadlib/private/Updater/ChangesMapper.php Outdated
Comment threadlib/public/AppFramework/Db/QBMapper.php

@ChristophWurstChristophWurst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ChangesMapper/Changes looks good now

Signed-off-by: jld3103 <jld3103yt@gmail.com>
@provokateurin
provokateurin merged commit c3b6b20 into masterApr 7, 2023
@provokateurin
provokateurin deleted the feature/type-mappers branch April 7, 2023 08:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewstechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@provokateurin@nickvergessen@ChristophWurst@miaulalala@come-nc