2021-04-14 00:01:03 +03:00
|
|
|
# Contributing to eBPF for Windows
|
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
We'd love your help with eBPF for Windows! Here are our contribution guidelines.
|
2021-04-14 00:01:03 +03:00
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
- [Code of Conduct](#code-of-conduct)
|
|
|
|
- [Bugs](#bugs)
|
|
|
|
- [New Features](#new-features)
|
|
|
|
- [Contributor License Agreement](#contributor-license-agreement)
|
|
|
|
- [Contributing Code](#contributing-code)
|
|
|
|
- [Tests](#tests)
|
|
|
|
|
|
|
|
## Code of Conduct
|
|
|
|
|
|
|
|
This project has adopted the [Microsoft Open Source Code of Conduct](https://opensource.microsoft.com/codeofconduct/).
|
|
|
|
For more information see the [Microsoft Code of Conduct FAQ](https://opensource.microsoft.com/codeofconduct/faq/)
|
|
|
|
or contact [opencode@microsoft.com](mailto:opencode@microsoft.com) with additional questions or comments.
|
|
|
|
|
|
|
|
## Bugs
|
|
|
|
|
|
|
|
### Did you find a bug?
|
|
|
|
|
|
|
|
First, **ensure the bug was not already reported** by searching on GitHub under
|
2021-04-14 00:01:03 +03:00
|
|
|
[Issues](https://github.com/microsoft/ebpf-for-windows/issues).
|
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
If you found a non-security related bug, you can help us by
|
|
|
|
[submitting a GitHub Issue](https://github.com/microsoft/ebpf-for-windows/issues/new).
|
|
|
|
The best bug reports provide a detailed description of the issue and step-by-step instructions
|
|
|
|
for reliably reproducing the issue. Even better, you can submit a Pull Request with a fix.
|
|
|
|
|
|
|
|
If you found a security issue, please **do not open a GitHub Issue**, and instead follow
|
|
|
|
[these instructions](docs/SECURITY.md).
|
2021-04-14 00:01:03 +03:00
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
### Did you write a patch that fixes a bug?
|
2021-04-14 00:01:03 +03:00
|
|
|
|
|
|
|
Fork the repo and make your changes.
|
|
|
|
Then open a new GitHub pull request with the patch.
|
|
|
|
|
|
|
|
* Ensure the PR description clearly describes the problem and solution.
|
|
|
|
Include the relevant issue number if applicable.
|
|
|
|
|
|
|
|
* Before submitting, please read the [Development Guide](docs/DevelopmentGuide.md)
|
|
|
|
to know more about coding conventions.
|
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
## New Features
|
|
|
|
|
|
|
|
You can request a new feature by [submitting a GitHub Issue](https://github.com/microsoft/ebpf-for-windows/issues/new).
|
|
|
|
|
|
|
|
If you would like to implement a new feature, please first
|
|
|
|
[submit a GitHub Issue](https://github.com/microsoft/ebpf-for-windows/issues/new) and
|
|
|
|
communicate your proposal so that the community can review and provide feedback. Getting
|
|
|
|
early feedback will help ensure your implementation work is accepted by the community.
|
|
|
|
This will also allow us to better coordinate our efforts and minimize duplicated effort.
|
|
|
|
|
|
|
|
## Contributor License Agreement
|
|
|
|
|
|
|
|
You will need to complete a Contributor License Agreement (CLA) for any code submissions.
|
|
|
|
Briefly, this agreement testifies that you are granting us permission to use the submitted
|
|
|
|
change according to the terms of the project's license, and that the work being submitted
|
|
|
|
is under appropriate copyright. You only need to do this once. For more information see
|
|
|
|
https://cla.opensource.microsoft.com/.
|
|
|
|
|
|
|
|
## Contributing Code
|
|
|
|
|
|
|
|
For all but the absolute simplest changes, first
|
|
|
|
[submit a GitHub Issue](https://github.com/microsoft/msquic/issues/new/choose) so that the
|
|
|
|
community can review and provide feedback. Getting early feedback will help ensure your work
|
|
|
|
is accepted by the community. This will also allow us to better coordinate our efforts and
|
|
|
|
minimize duplicated effort.
|
|
|
|
|
|
|
|
If you would like to contribute, first identify the scale of what you would like to contribute.
|
|
|
|
If it is small (grammar/spelling or a bug fix) feel free to start working on a fix. If you are
|
|
|
|
submitting a feature or substantial code contribution, please discuss it with the maintainers and
|
|
|
|
ensure it follows the product roadmap. You might also read these two blogs posts on contributing
|
|
|
|
code: [Open Source Contribution Etiquette](http://tirania.org/blog/archive/2010/Dec-31.html) by Miguel de Icaza and
|
|
|
|
[Don't "Push" Your Pull Requests](https://www.igvita.com/2011/12/19/dont-push-your-pull-requests/) by Ilya Grigorik.
|
|
|
|
All code submissions will be rigorously reviewed and tested by the maintainers, and only those that meet
|
|
|
|
the bar for both quality and design/roadmap appropriateness will be merged into the source.
|
2021-04-14 00:01:03 +03:00
|
|
|
|
2021-04-14 21:45:32 +03:00
|
|
|
For all new Pull Requests the following rules apply:
|
|
|
|
- Existing tests should continue to pass.
|
|
|
|
- Tests need to be provided for every bug/feature that is completed.
|
|
|
|
- Documentation needs to be provided for every feature that is end-user visible.
|