|
| 1 | +# Welcome to gtk4-nvidia-monitor-rust contributing guide |
| 2 | + |
| 3 | +Thank you for investing your time in contributing to our project! |
| 4 | + |
| 5 | +Read our [Code of Conduct](CODE_OF_CONDUCT.md) to keep our community approachable and respectable. |
| 6 | + |
| 7 | +In this guide you will get an overview of the contribution workflow from opening an issue, creating a PR, reviewing, and merging the PR. |
| 8 | + |
| 9 | +## New contributor guide |
| 10 | + |
| 11 | +To get an overview of the project, read the [README](README.md). Here are some resources to help you get started making contributions: |
| 12 | + |
| 13 | +- [Bug Issue Template](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues/new?assignees=&labels=&template=bug_report.md&title=) |
| 14 | +- [Feature Request Issue Template](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues/new?assignees=&labels=&template=feature_request.md&title=) |
| 15 | +- [Pull Request (PR) Template]() |
| 16 | + |
| 17 | +## Getting started |
| 18 | + |
| 19 | +Check out the open issues [here](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues) |
| 20 | + |
| 21 | +Issues tagged "good first issue" and "help wanted" are your best bet - otherwise open a feature request issue and we can discuss changes! |
| 22 | + |
| 23 | +### Issues |
| 24 | + |
| 25 | +#### Create a new issue |
| 26 | + |
| 27 | +If you spot a bug or want a new feature, [search if an issue already exists](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues). If a related issue doesn't exist, you can open a new issue using a relevant [issue form](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues/new/choose). |
| 28 | + |
| 29 | +#### Solve an issue |
| 30 | + |
| 31 | +Scan through our [existing issues](https://github.com/derenv/gtk4-nvidia-monitor-rust/issues) to find one that interests you. You can narrow down the search using `labels` as filters. As a general rule, we don’t assign issues to anyone. If you find an issue to work on, you are welcome to open a PR with a fix. |
| 32 | + |
| 33 | +### Make Changes |
| 34 | + |
| 35 | +#### Make changes in the UI |
| 36 | + |
| 37 | +Click **Make a contribution** at the bottom of any docs page to make small changes such as a typo, sentence fix, or a broken link. This takes you to the `.md` file where you can make your changes and [create a pull request](#pull-request) for a review. |
| 38 | + |
| 39 | +#### Make changes locally |
| 40 | + |
| 41 | +1. Fork the repository. |
| 42 | +- Using GitHub Desktop: |
| 43 | + - [Getting started with GitHub Desktop](https://docs.github.com/en/desktop/installing-and-configuring-github-desktop/getting-started-with-github-desktop) will guide you through setting up Desktop. |
| 44 | + - Once Desktop is set up, you can use it to [fork the repo](https://docs.github.com/en/desktop/contributing-and-collaborating-using-github-desktop/cloning-and-forking-repositories-from-github-desktop)! |
| 45 | + |
| 46 | +- Using the command line: |
| 47 | + - [Fork the repo](https://docs.github.com/en/github/getting-started-with-github/fork-a-repo#fork-an-example-repository) so that you can make your changes without affecting the original project until you're ready to merge them. |
| 48 | + |
| 49 | +2. Install or update to **Rust 1.66.0**. All dependancies are in the Cargo.toml file, and will be compiled when you build the project |
| 50 | + |
| 51 | +3. Create a working branch and start with your changes! |
| 52 | + |
| 53 | +### Commit your update |
| 54 | + |
| 55 | +Commit the changes once you are happy with them. |
| 56 | + |
| 57 | +### Pull Request |
| 58 | + |
| 59 | +When you're finished with the changes, create a pull request, also known as a PR. |
| 60 | +- Fill the "Ready for review" template so that we can review your PR. This template helps reviewers understand your changes as well as the purpose of your pull request. |
| 61 | +- Don't forget to [link PR to issue](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) if you are solving one. |
| 62 | +- Enable the checkbox to [allow maintainer edits](https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/allowing-changes-to-a-pull-request-branch-created-from-a-fork) so the branch can be updated for a merge. |
| 63 | +Once you submit your PR, a team member will review your proposal. We may ask questions or request additional information. |
| 64 | +- We may ask for changes to be made before a PR can be merged, either using [suggested changes](https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/incorporating-feedback-in-your-pull-request) or pull request comments. You can apply suggested changes directly through the UI. You can make any other changes in your fork, then commit them to your branch. |
| 65 | +- As you update your PR and apply changes, mark each conversation as [resolved](https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/commenting-on-a-pull-request#resolving-conversations). |
| 66 | +- If you run into any merge issues, checkout this [git tutorial](https://github.com/skills/resolve-merge-conflicts) to help you resolve merge conflicts and other issues. |
| 67 | + |
| 68 | +### Your PR is merged! |
| 69 | + |
| 70 | +Congratulations :tada::tada: |
| 71 | + |
| 72 | +Once your PR is merged, your contributions will be part of next release. |
0 commit comments