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

Add myself as iOS co-maintainer #2451

Merged
merged 1 commit into from
Aug 31, 2022

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented Aug 31, 2022

Hey @francesca64, you noted in #2428 (comment) that you no longer have access to iOS devices (also sorry to hear you're unemployed if you're affected by that in other ways!).

I'm quite proficient in Objective-C at this point, and have access to an iOS device I can test on, and while that is of course very far from "knowing" UIKit, I think it would make sense for me to take on co-responsibility of the iOS platform. I'll probably be limiting myself to stuff relating to Objective-C to begin with, e.g. I wouldn't really feel comfortable tackling #2144.

@madsmtm madsmtm added DS - ios S - maintenance Repaying technical debt labels Aug 31, 2022
@madsmtm
Copy link
Member Author

madsmtm commented Aug 31, 2022

Concretely, I'd really like to get #2427 merged at some point, and it sounds like you won't really be able to review/test it, so I think it makes sense to have a discussion here of whether you would be comfortable with me merging such changes to iOS?

Copy link
Member

@francesca64 francesca64 left a comment

Choose a reason for hiding this comment

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

I totally trust you! Feel free to merge anything you'd like, and thanks a ton for stepping up.

@madsmtm
Copy link
Member Author

madsmtm commented Aug 31, 2022

Wonderful, thanks!

@madsmtm madsmtm merged commit 66aa6c9 into rust-windowing:master Aug 31, 2022
@madsmtm madsmtm deleted the myself-ios-maintainer branch August 31, 2022 23:45
@madsmtm madsmtm mentioned this pull request Sep 1, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DS - ios S - maintenance Repaying technical debt
Development

Successfully merging this pull request may close these issues.

2 participants