2018-11-10 21:56:40 +11:00
|
|
|
# Winit Contributing Guidelines
|
|
|
|
|
|
|
|
## Scope
|
2019-04-14 08:57:08 +10:00
|
|
|
[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.
|
2018-11-10 21:56:40 +11:00
|
|
|
|
|
|
|
|
|
|
|
## Reporting an issue
|
|
|
|
|
2018-12-25 05:08:33 +11:00
|
|
|
When reporting an issue, in order to help the maintainers understand what the problem is, please make
|
2018-11-10 21:56:40 +11:00
|
|
|
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
|
2018-12-25 05:08:33 +11:00
|
|
|
- you left comments in your code explaining any part that is not straightforward, so that the
|
2018-11-10 21:56:40 +11:00
|
|
|
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
|
2019-04-14 08:57:08 +10:00
|
|
|
- if your PR affects the platform compatibility of one or more features or adds another feature, the
|
2019-04-15 02:05:37 +10:00
|
|
|
relevant sections in [`FEATURES.md`](https://github.com/rust-windowing/winit/blob/master/FEATURES.md#features)
|
|
|
|
should be updated.
|
2018-11-10 21:56:40 +11:00
|
|
|
|
|
|
|
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`: is a main maintainer for this platform
|
|
|
|
- `R`: can review code for this platform
|
|
|
|
- `T`: has the ability of testing the platform
|
|
|
|
- ` `: knows nothing of this platform
|
|
|
|
|
|
|
|
| Platform | Windows | macOS | X11 | Wayland | Android | iOS | Emscripten |
|
|
|
|
| :--- | :---: | :---: | :---: | :---: | :---: | :---: | :---: |
|
2018-12-25 05:08:33 +11:00
|
|
|
| @francesca64 | R | M | M | | M | R | |
|
2018-11-10 21:56:40 +11:00
|
|
|
| @mitchmindtree | T | | T | T | | | |
|
|
|
|
| @Osspial | M | | T | T | T | | T |
|
|
|
|
| @vberger | | | T | M | | | |
|
2018-11-13 13:49:15 +11:00
|
|
|
| @mtak- | | T | | | T | M | |
|