mirror of
https://github.com/italicsjenga/winit-sonoma-fix.git
synced 2024-12-23 22:01:31 +11:00
e087ebd1c7
This includes de-listing @francesca64 from the table, as I suggested. I realize that this is a controversial decision, and it's not a decision I make lightly, but I believe I have justification for doing so: I contacted @francesca64 last month asking her about her inactivity as a maintainer on this project. She replied on March 26th as follows. For the sake of her privacy, I've removed removed certain sections of her response, as it contains some personal details that I'm not comfortable sharing with the world at large without her explicit permission. > Hello! Thanks for reaching out!❤ > In short, I've moved on. <removed> ...in November, I was hired by a > company that recently started using Rust. I'm very happily building > infrastructure there! > I'm simply not interested in spending more than 8 hours a day > programming. You couldn't even pay me to do it! My time is best spent > going on adventures with my beloved. > I'll still be active in the Rust ecosystem, but only insofar as the > company I work for is. We use winit on iOS, Android, and (to a limited > extent) macOS, so I'll work on those backends as needed. > Thank you for taking care of winit. I hope you're taking care of > yourself too; <removed>. I don't begrudge her for her decision, and others shouldn't either - I firmly believe that, as this is unpaid, volunteer work, everybody should have the right to move on when they decide they no longer have the time or will to contribute. The exact impliciation of this in regards to her status as maintainer is open to interpretation. I would argue that this means, should she in her work stumble upon an issue in any of her listed backends, she would be willing to submit PRs addressing those issues. However, it also means that she is not able to put in the time to be active as a maintainer on those platforms, or review PRs and issues for those platforms. On March 28th I responded to her email as follows: > Hey, thanks for responding. > <removed, personal details> > In the meanwhile, there are still a few loose ends from your time as > maintainer that I'd like to get cleaned up. It's okay if you aren't > going to be spending much time on Winit, but there's still a broad > assumption that you're able to review PRs for them and that doesn't seem > to be the case. Would you be able to do a couple things to ease the > transition to whoever next takes over the macOS, X11, and Android > backends? > 1) Submit a PR downgrading yourself from maintainer for macOS, X11, and > Android, so somebody else can more actively take them over. > 2) Post your WIP macOS backend for EL2.0 as well as the issues it > currently has, so whoever next maintains macOS can finish it. > Going forward, I'd like to reach out to the broader Rust community and > find more active maintainers so Winit can get to 1.0 and I can mostly > move on from it. I recieved no response to that email. On April 4th, I followed up on that email: > If you aren't able to act as a maintainer for Winit, and aren't able to > submit a PR updating your official status as maintainer to reflect > reality, would you mind if I submitted a PR removing you as maintainer? > That's not something I want to do since it's a bad image for me, a bad > image for Winit, and sets an extremely uncomfortable precedent, but I'd > like to start more aggressive outreach to ensure each backend is less > dependent on one specific person and I don't want to see new > contributors pinging you for help when you're unable to provide it. > If you don't reply by the 11th that's the path I'm going to take, but I > consider it the nuclear option and I want to avoid invoking it if at all > possible. Up to this date (April 13th), I have recieved no response. Given the amount of time I've given her to respond, as well as her lack of response, I believe we have the justification to remove her from the table. Should she show back up again, any clarifications on her status would be welcome, and she is welcome to submit a PR re-listing herself on the table with a more accurate description of her current contributor status. However, once we begin the contributor marketing push discussed in #830, I don't want new contributors to attempt to ask her questions on the macOS, X11, or Android backends when she isn't able to give a response. ----------------------------------------------------------------------- This PR also introduces HALL_OF_CHAMPIONS.md, which commends the efforts of former maintainers that have contributed greatly to the Winit project. This wasn't discussed previously, but I think it's important to recognize the people that brought us to where we are today. It currently lists @tomaka and @francesca64, as they are the two individuals I'm aware of that both deserve such recognition and no longer actively contribute to Winit, but if there's anybody I missed feel free to suggest them and a blurb describing their work.
55 lines
3.1 KiB
Markdown
55 lines
3.1 KiB
Markdown
# Winit Contributing Guidelines
|
|
|
|
## Scope
|
|
[See `FEATURES.md`](./FEATURES.md). When requesting or implementing a new Winit feature, you should
|
|
consider whether or not it's directly related to window creation or input handling. If it isn't, it
|
|
may be worth creating a separate crate that extends Winit's API to add that functionality.
|
|
|
|
|
|
## Reporting an issue
|
|
|
|
When reporting an issue, in order to help the maintainers understand what the problem is, please make
|
|
your description of the issue as detailed as possible:
|
|
|
|
- if it is a bug, please provide clear explanation of what happens, what should happen, and how to
|
|
reproduce the issue, ideally by providing a minimal program exhibiting the problem
|
|
- if it is a feature request, please provide a clear argumentation about why you believe this feature
|
|
should be supported by winit
|
|
|
|
## Making a pull request
|
|
|
|
When making a code contribution to winit, before opening your pull request, please make sure that:
|
|
|
|
- you tested your modifications on all the platforms impacted, or if not possible detail which platforms
|
|
were not tested, and what should be tested, so that a maintainer or another contributor can test them
|
|
- you updated any relevant documentation in winit
|
|
- you left comments in your code explaining any part that is not straightforward, so that the
|
|
maintainers and future contributors don't have to try to guess what your code is supposed to do
|
|
- your PR adds an entry to the changelog file if the introduced change is relevant to winit users
|
|
- if your PR affects the platform compatibility of one or more features or adds another feature, the
|
|
relevant sections in [`FEATURES.md`](https://github.com/rust-windowing/winit/blob/master/FEATURES.md#features)
|
|
should be updated.
|
|
|
|
Once your PR is open, you can ask for review by a maintainer of your platform. Winit's merging policy
|
|
is that a PR must be approved by at least two maintainers of winit before being merged, including
|
|
at least a maintainer of the platform (a maintainer making a PR themselves counts as approving it).
|
|
|
|
## Maintainers & Testers
|
|
|
|
Winit is managed by several people, each with their specialities, and each maintaining a subset of the
|
|
backends of winit. As such, depending on your platform of interest, your contacts will be different.
|
|
|
|
This table summarizes who can be contacted in which case, with the following legend:
|
|
|
|
- `M` - Maintainer: is a main maintainer for this platform
|
|
- `C` - Collaborator: can review code and address issues on this platform
|
|
- `T` - Tester: has the ability of testing the platform
|
|
- ` `: knows nothing of this platform
|
|
|
|
| Platform | Windows | macOS | X11 | Wayland | Android | iOS | Emscripten |
|
|
| :--- | :---: | :---: | :---: | :---: | :---: | :---: | :---: |
|
|
| @mitchmindtree | T | | T | T | | | |
|
|
| @Osspial | M | | T | T | T | | T |
|
|
| @vberger | | | T | M | | | |
|
|
| @mtak- | | T | | | T | M | |
|