Skip to content

Commit 07e3a27

Browse files
committed
fix CI
1 parent b589261 commit 07e3a27

File tree

2 files changed

+39
-36
lines changed

2 files changed

+39
-36
lines changed

.devcontainer/Dockerfile.dev

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,8 @@ COPY ci /opt/ci
1414

1515
RUN apt update && apt install -y wget \
1616
ninja-build \
17-
libcurl4-openssl-dev
17+
libcurl4-openssl-dev \
18+
markdownlint
1819

1920
RUN cd /opt/ci && bash setup_cmake.sh
2021
RUN cd /opt/ci && bash setup_ci_environment.sh

CONTRIBUTING.md

Lines changed: 37 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -62,86 +62,88 @@ bazel-bin/examples/simple/example_simple
6262

6363
### DevContainer Setup for Project
6464

65-
This guide provides instructions on how to set up and use the development
66-
container (`devcontainer`) environment to streamline testing and development
67-
for this project. With the DevContainer, you can work in a consistent environment
65+
This guide provides instructions on how to set up and use the development
66+
container (`devcontainer`) environment to streamline testing and development
67+
for this project. With the DevContainer, you can work in a consistent environment
6868
configured with all the necessary dependencies and tools.
6969

7070
#### Prerequisites
7171

7272
Before getting started, ensure you have the following installed:
7373

74-
1. **Docker**: DevContainers require Docker for containerization.
75-
2. **Visual Studio Code (VSCode)** with the **Remote - Containers** extension.
74+
* **Docker**: DevContainers require Docker for containerization.
75+
* **Visual Studio Code (VSCode)** with the **Remote - Containers** extension.
7676

7777
#### Getting Started
7878

79-
1. **Open the Project in DevContainer**:
79+
* **Open the Project in DevContainer**:
8080

81-
Open the project in VSCode. When prompted to "Reopen in Container," select
82-
this option. If you’re not prompted, you can manually open the container by
83-
selecting **Remote-Containers: Reopen in Container** from the command palette
81+
Open the project in VSCode. When prompted to "Reopen in Container," select
82+
this option. If you’re not prompted, you can manually open the container by
83+
selecting **Remote-Containers: Reopen in Container** from the command palette
8484
(`F1` or `Ctrl+Shift+P`).
8585

86-
2. **Container Setup**:
86+
* **Container Setup**:
8787

88-
The DevContainer environment will automatically build based on the configuration
89-
files provided (e.g., `.devcontainer/devcontainer.json`). This setup will install
88+
The DevContainer environment will automatically build based on the configuration
89+
files provided (e.g., `.devcontainer/devcontainer.json`). This setup will install
9090
required dependencies, tools, and environment variables needed for the project.
9191

9292
#### Available Commands
9393

94-
Once inside the DevContainer, you can use the following commands to run tests
94+
Once inside the DevContainer, you can use the following commands to run tests
9595
and CI workflows.
9696

9797
##### 1. Run Tests with Bazelisk
9898

9999
To run tests with Bazelisk using specific compilation options, use:
100100

101101
```bash
102-
bazelisk-linux-amd64 test --copt=-DENABLE_LOGS_PREVIEW --test_output=errors --cache_test_results=no --copt=-DENABLE_TEST //exporters/otlp/...
102+
bazelisk-linux-amd64 test --copt=-DENABLE_LOGS_PREVIEW
103+
--test_output=errors --cache_test_results=no --copt=-DENABLE_TEST //exporters/otlp/...
103104
```
104105

105106
###### Command Breakdown
106107

107-
- `--copt=-DENABLE_LOGS_PREVIEW`: Enables preview logs.
108-
- `--test_output=errors`: Shows only the errors in the test output.
109-
- `--cache_test_results=no`: Forces Bazel to re-run tests without caching.
110-
- `--copt=-DENABLE_TEST`: Enables testing capabilities for the target code.
111-
- `//exporters/otlp/...`: Specifies the test target path.
108+
* `--copt=-DENABLE_LOGS_PREVIEW`: Enables preview logs.
109+
* `--test_output=errors`: Shows only the errors in the test output.
110+
* `--cache_test_results=no`: Forces Bazel to re-run tests without caching.
111+
* `--copt=-DENABLE_TEST`: Enables testing capabilities for the target code.
112+
* `//exporters/otlp/...`: Specifies the test target path.
112113

113114
##### 2. Run CI Script
114115

115-
You can also run the CI script provided to perform testing with the following command as an
116+
You can also run the CI script provided to perform testing with the
117+
following command as an
116118
example:
117119

118120
```bash
119121
bash ci/do_ci.sh cmake.exporter.otprotocol.test
120122
```
121123

122-
This command initiates the CI pipeline, executing tests specifically for the
124+
This command initiates the CI pipeline, executing tests specifically for the
123125
**cmake.exporter.otprotocol** module.
124126

125127
#### Troubleshooting
126128

127129
If you encounter issues:
128130

129-
- **Rebuild the DevContainer**: From the command palette, run
131+
* **Rebuild the DevContainer**: From the command palette, run
130132
**Remote-Containers: Rebuild Container** to reinitialize the environment.
131-
- **Check Bazelisk and CI Script Logs**: Inspect logs for any configuration or
133+
* **Check Bazelisk and CI Script Logs**: Inspect logs for any configuration or
132134
dependency issues.
133135

134136
#### Additional Notes
135137

136-
- You can adjust compiler options (`--copt`) as needed to test additional flags
138+
* You can adjust compiler options (`--copt`) as needed to test additional flags
137139
or enable/disable specific features.
138-
- The test results will be displayed in the terminal within the DevContainer for
140+
* The test results will be displayed in the terminal within the DevContainer for
139141
easy debugging.
140142

141143
#### Resources
142144

143-
- **Bazelisk Documentation**: [https://github.com/bazelbuild/bazelisk](https://github.com/bazelbuild/bazelisk)
144-
- **VSCode DevContainer Documentation**: [https://code.visualstudio.com/docs/remote/containers](https://code.visualstudio.com/docs/remote/containers)
145+
* **Bazelisk Documentation**: [https://github.com/bazelbuild/bazelisk](https://github.com/bazelbuild/bazelisk)
146+
* **VSCode DevContainer Documentation**: [https://code.visualstudio.com/docs/remote/containers](https://code.visualstudio.com/docs/remote/containers)
145147

146148
## Pull Requests
147149

@@ -269,16 +271,16 @@ the C++ repository.
269271

270272
* [OpenTelemetry
271273
Specification](https://github.com/open-telemetry/opentelemetry-specification)
272-
* The OpenTelemetry Specification describes the requirements and expectations
273-
of for all OpenTelemetry implementations.
274+
* The OpenTelemetry Specification describes the requirements and expectations
275+
of for all OpenTelemetry implementations.
274276

275277
* Read through the OpenTelemetry C++ documentation
276-
* The
277-
[API](https://opentelemetry-cpp.readthedocs.io/en/latest/api/api.html)
278-
and
279-
[SDK](https://opentelemetry-cpp.readthedocs.io/en/latest/sdk/sdk.html)
280-
documentation provides a lot of information on what the classes and their
281-
functions are used for.
278+
* The
279+
[API](https://opentelemetry-cpp.readthedocs.io/en/latest/api/api.html)
280+
and
281+
[SDK](https://opentelemetry-cpp.readthedocs.io/en/latest/sdk/sdk.html)
282+
documentation provides a lot of information on what the classes and their
283+
functions are used for.
282284

283285
Please contribute! You’re welcome to add more information if you come across any
284286
helpful resources.

0 commit comments

Comments
 (0)