d0850e95fb
Changed the name of the unit tests CI job to accuratly reflect that both AMD64 and ARM64 tests are executed. Modified the contribution guide to specify that contributors should work inside their own fork of the project.
80 lines
6.0 KiB
Markdown
80 lines
6.0 KiB
Markdown
# Contributing to Veilid
|
|
Before you get started, please review our [Code of Conduct](./code_of_conduct.md). We're here to make things better and we cannot do that without treating each other with respect.
|
|
|
|
|
|
## Code Contributions
|
|
To begin crafting code to contribute to the Veilid project, first set up a [development environment](./DEVELOPMENT.md). [Fork] and clone the project into your workspace; check out a new local branch and name it in a way that describes the work being done. This is referred to as a [feature branch].
|
|
|
|
Some contributions might introduce changes that are incompatible with other existing nodes. In this case it is recommended to also set a development network *Guide Coming Soon*.
|
|
|
|
Once you have added your new function or addressed a bug, test it locally to ensure it's working as expected. If needed, test your work in a development network with more than one node based on your code. Once you're satisfied your code works as intended and does not introduce negative results or new bugs, follow the merge requests section below to submit your work for maintainer review.
|
|
|
|
We try to consider all merge requests fairly and with attention deserving to those willing to put in time and effort, but if you do not follow these rules, your contribution will be closed. We strive to ensure that the code joining the main branch is written to a high standard.
|
|
|
|
|
|
### Code Contribution Do's & Don'ts:
|
|
|
|
Keeping the following in mind gives your contribution the best chance of landing!
|
|
|
|
#### <u>Merge Requests</u>
|
|
|
|
* **Do** start by [forking] the project.
|
|
* **Do** create a [feature branch] to work on instead of working directly on `main`. This helps to:
|
|
* Protect the process.
|
|
* Ensures users are aware of commits on the branch being considered for merge.
|
|
* Allows for a location for more commits to be offered without mingling with other contributor changes.
|
|
* Allows contributors to make progress while a MR is still being reviewed.
|
|
* **Do** follow the [50/72 rule] for Git commit messages.
|
|
* **Do** target your merge request to the **main branch**.
|
|
* **Do** specify a descriptive title to make searching for your merge request easier.
|
|
* **Do** list [verification steps] so your code is testable.
|
|
* **Do** reference associated issues in your merge request description.
|
|
* **Don't** leave your merge request description blank.
|
|
* **Don't** abandon your merge request. Being responsive helps us land your code faster.
|
|
* **Don't** submit unfinished code.
|
|
|
|
|
|
|
|
## Contributions Without Writing Code
|
|
There are numerous ways you can contribute to the growth and success of the Veilid project without writing code:
|
|
|
|
- If you want to submit merge requests, begin by [forking] the project and checking out a new local branch. Name your new branch in a way that describes the work being done. This is referred to as a [feature branch].
|
|
- Submit bugs as well as feature/enhancement requests. Letting us know you found a bug, have an idea for a new feature, or see a way we can enhance existing features is just as important and useful as writing the code related to those things. Send us detailed information about your issue or idea:
|
|
- Features/Enhancements: Describe your idea. If you're able to, sketch out a diagram or mock-up.
|
|
- Bugs: Please be sure to include the expected behavior, the observed behavior, and steps to reproduce the problem. Please be descriptive about the environment you've installed your node or application into.
|
|
- [Help other users with open issues]. Sometimes all an issue needs is a little conversation to clear up a process or misunderstanding. Please keep the [Code of Conduct](./code_of_conduct.md) in mind.
|
|
- Help other contributors test recently submitted merge requests. By pulling down a merge request and testing it, you can help validate new code contributions for stability and quality.
|
|
- Report a security or privacy vulnerability. Please let us know if you find ways in which Veilid could handle security and/or privacy in a different or better way. Surely let us know if you find broken or otherwise flawed security and/or privacy functions. You can report these directly to security@veilid.org.
|
|
- Add or edit documentation. Documentation is a living and evolving library of knowledge. As such, care, feeding, and even pruning is needed from time to time. If you're a non-native english speaker, you can help by replacing any ambiguous idioms, metaphors, or unclear language that might make our documentation hard to understand.
|
|
|
|
|
|
#### <u>Bug Fixes</u>
|
|
* **Do** include reproduction steps in the form of [verification steps].
|
|
* **Do** link to any corresponding issues in your commit description.
|
|
|
|
## Bug Reports
|
|
|
|
When reporting Veilid issues:
|
|
* **Do** write a detailed description of your bug and use a descriptive title.
|
|
* **Do** include reproduction steps, stack traces, and anything that might help us fix your bug.
|
|
* **Don't** file duplicate reports. Search open issues for similar bugs before filing a new report.
|
|
* **Don't** attempt to report issues on a closed PR. New issues should be openned against the `main` branch.
|
|
|
|
Please report vulnerabilities in Veilid directly to security@veilid.org.
|
|
|
|
If you're looking for more guidance, talk to other Veilid contributors on the [Veilid Discord].
|
|
|
|
**Thank you** for taking the few moments to read this far! Together we will build something truely remarkable.
|
|
|
|
|
|
|
|
This contributor guide is inspired by the contribution guidelines of the [Metasploit Framework](https://github.com/rapid7/metasploit-framework/blob/master/CONTRIBUTING.md) project found on GitHub.
|
|
|
|
[Help other users with open issues]:https://gitlab.com/veilid/veilid/-/issues
|
|
[50/72 rule]:http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
|
|
[feature branch]:https://docs.gitlab.com/ee/gitlab-basics/feature_branch_workflow.html
|
|
[forking]:https://docs.gitlab.com/ee/user/project/repository/forking_workflow.html
|
|
[fork]:https://docs.gitlab.com/ee/user/project/repository/forking_workflow.html
|
|
[verification steps]:https://docs.gitlab.com/ee/user/markdown.html#task-lists
|
|
[Veilid Discord]:https://discord.gg/DmEGxyA87N
|