Skip to content

Commit 15be817

Browse files
RichardChukwujaydelucatrask
authored
Improve Contributing Guide (#1680)
Co-authored-by: Jay DeLuca <[email protected]> Co-authored-by: Trask Stalnaker <[email protected]>
1 parent b079b13 commit 15be817

File tree

1 file changed

+50
-10
lines changed

1 file changed

+50
-10
lines changed

CONTRIBUTING.md

Lines changed: 50 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,38 @@
1-
## Contributing
1+
# Contributing
22

3-
Pull requests for bug fixes are always welcome!
3+
Welcome to the OpenTelemetry Java Contrib Repository!
4+
5+
## Introduction
6+
7+
This repository focuses on providing tools and utilities for Java-based observability, such as remote JMX metric gathering and reporting. We’re excited to have you here! Whether you’re fixing a bug, adding a feature, or suggesting an idea, your contributions are invaluable.
48

59
Before submitting new features or changes to current functionality, it is recommended to first
610
[open an issue](https://github.com/open-telemetry/opentelemetry-java-contrib/issues/new)
711
and discuss your ideas or propose the changes you wish to make.
812

9-
### Building
13+
Questions? Ask in the OpenTelemetry [java channel](https://cloud-native.slack.com/archives/C014L2KCTE3)
14+
15+
Pull requests for bug fixes are always welcome!
16+
17+
## Pre-requisites
18+
19+
To work with this repository, ensure you have:
20+
21+
### Tools:
22+
23+
Java 17 or higher
24+
25+
### Platform Notes:
26+
27+
macOS/Linux: Ensure JAVA_HOME is set correctly.
28+
29+
## Workflow
30+
31+
1. Fork the repository
32+
2. Clone locally
33+
3. Create a branch before working on an issue
34+
35+
## Local Run/Build
1036

1137
In order to build and test this whole repository you need JDK 11+.
1238

@@ -22,21 +48,35 @@ the Sonatype OSS snapshots repository at `https://oss.sonatype.org/content/repos
2248
Building using Java 11+:
2349

2450
```bash
25-
java -version
51+
$ java -version
52+
```
53+
54+
```bash
55+
$ ./gradlew assemble
2656
```
2757

58+
## Testing
59+
2860
```bash
29-
./gradlew assemble
61+
$ ./gradlew test
3062
```
3163

32-
### Style guide
64+
### Some modules have integration tests
3365

34-
See
35-
the [Style guide](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/style-guideline.md)
36-
from the opentelemetry-java-instrumentation repository.
66+
```
67+
$ ./gradlew integrationTest
68+
```
69+
70+
Follow the Java Instrumentation [Style Guide](https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/docs/contributing/style-guideline.md) from the opentelemetry-java-instrumentation repository.
3771

38-
### Gradle conventions
72+
Failure? Check logs for errors or mismatched dependencies.
73+
74+
## Gradle conventions
3975

4076
- Use kotlin instead of groovy
4177
- Plugin versions should be specified in `settings.gradle.kts`, not in individual modules
4278
- All modules use `plugins { id("otel.java-conventions") }`
79+
80+
## Further Help
81+
82+
Join [#otel-java](https://cloud-native.slack.com/archives/C014L2KCTE3) on OpenTelemetry Slack

0 commit comments

Comments
 (0)