* Add CODEOWNERS file This makes it very clear when you're stepping down from the post as a maintainer, and makes it clear for users who is expected to review their PR * Fix grammar * Make @kchibisov receive pings for the X11 platform * Fix typo Co-authored-by: Kirill Chibisov <contact@kchibisov.com>
3.5 KiB
Winit Contributing Guidelines
Scope
See 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:
-
your patch builds with Winit's minimal supported rust version - Rust 1.57.0.
-
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.
You needn't worry about the added entry causing conflicts, the maintainer that merges the PR will handle those for you when merging (see below).
-
if your PR affects the platform compatibility of one or more features or adds another feature, the relevant sections in
FEATURES.md
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).
Once your PR is deemed ready, the merging maintainer will take care of resolving conflicts in
CHANGELOG.md
(but you must resolve other conflicts yourself). Doing this requires that you check the
"give contributors write access to the branch" checkbox when creating the PR.
Maintainers & Testers
The current maintainers are listed in the CODEOWNERS file.
If you are interested in being pinged when testing is needed for a certain platform, please add yourself to the Testers and Contributors table!
Making a new release
If you believe a new release is warranted, you can make a pull-request with:
- An updated version number (remember to change the version everywhere it is used).
- A new section in the changelog (below the
# Unreleased
section).
This gives contributors an opportunity to squeeze in an extra PR or two that they feel is valuable enough to warrant blocking the release a little.
Once the PR is merged, a maintainer will create a new tag matching the version name (e.g. v0.26.1
),
and a CI job will automatically release the new version. Remember that the release date in the
changelog must be kept in check with the actual release date.