Skip to content

Commit 8174a9c

Browse files
authored
Merge pull request #87 from lightstep/add-contributing-guide
Add CONTRIBUTING.md guide
2 parents 0c16850 + c7adbd3 commit 8174a9c

File tree

1 file changed

+67
-0
lines changed

1 file changed

+67
-0
lines changed

CONTRIBUTING.md

Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
# LightStep Community Contributing Guide
2+
3+
First, 🎉 **thanks for contributing!** 🎉
4+
5+
## Issues
6+
7+
You're encouraged to log issues for any questions or problems you might have. When in doubt, log an issue. The exception to this rule is [security disclosures](#reporting-security-issues).
8+
9+
Generally speaking, the more context you can provide, the better. Please add information such as what **version** you're using, **stack traces** and/or **logs** (to the extent that you're able to share them), and whatever else you think may be relevant. Project maintainers may ask for additional clarification, logs, and other pertinent metadata before we can address your issue.
10+
11+
For bug submission, we especially appreciate **details on how to reproduce the bug** to the extent you're able to provide them, e.g., an isolated repo or [gist](https://gist.github.com).
12+
13+
### Reporting Security Issues
14+
15+
If you find a security issue, please **do not** file a public issue for it. Instead, send your report to us privately at [security@lightstep.com](mailto:security@lightstep.com).
16+
17+
## Contributions
18+
19+
All contributions big and small are welcome, from typo corrections to bug fixes to suggested improvements!
20+
21+
Any changes to project resources in this repository must be made through a pull request. This includes, but is not limited to, changes affecting:
22+
23+
- Documentation
24+
- Source code
25+
- Binaries
26+
- Sample projects or other examples
27+
28+
No pull request can be merged without at least one review from a maintainer.
29+
30+
By default, contributions are accepted once no committers object to the PR. Specific contributors may be suggested or required to review a pull request based on repository settings.
31+
32+
In the event of objections or disagreement, everyone involved should seek to arrive at a consensus around the expressed objections. These can take the form of addressing concerns through changes, compromising around the change, or withdrawing it entirely.
33+
34+
## Development
35+
36+
### Testing
37+
38+
To run the tests:
39+
40+
```
41+
make test
42+
```
43+
44+
### Style Guide
45+
46+
To check for linting errors:
47+
48+
```
49+
make lint
50+
```
51+
52+
## Submitting a Pull Request
53+
54+
_Note that this section should be treated as a general guideline and replaced with language/repo specific instructions_
55+
56+
1. [Fork the repository.](https://help.github.com/en/github/getting-started-with-github/fork-a-repo)
57+
1. Create a new branch.
58+
1. Add tests for your change.
59+
1. [Run the tests](#testing) to make sure that they don't already pass. If they do (and you're not backfilling test coverage), please modify them.
60+
1. Implement the change such that your new tests pass.
61+
1. Make sure that your code conforms to the [style guide](#style-guide).
62+
1. [Commit and push your changes.](https://guides.github.com/introduction/flow/)
63+
1. [Submit your pull request.](https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests)
64+
1. Adjust your pull request based on feedback.
65+
1. Get it merged! 🎉
66+
67+
We're happy to help with any questions you may have on the git or GitHub side, e.g., how to push a branch to your fork. Just create an issue and we'll try to help answer them :)

0 commit comments

Comments
 (0)