docs: fix broken relative links in experimental docs - #7046
Conversation
Three relative links in the experimental docs pointed at paths that do not exist: - experimental/CHANGELOG.md linked to doc/upgrade-to-2.x.md, but the upgrade guide lives at the repository root, so from experimental/ it needs ../. - sdk-logs/README.md linked to ./src/config.ts, which no longer exists. The default LoggerConfig is now defined in ./src/config/LoggerConfigurators.ts. - exporter-trace-otlp-grpc/README.md linked to ./test/certs/regenerate.sh, but that package has no test/certs directory. The script lives in otlp-grpc-exporter-base, which the package already depends on. Signed-off-by: Mihirsinh Chavda <mihhhir08@gmail.com>
|
Welcome, contributor! Thank you for your contribution to opentelemetry-js. Important reminders:
|
Pull request dashboard statusWaiting on maintainers · refreshed 2026-09-08 08:38 UTC Merge when ready. Status above doesn't look right?
|
…ved file Assisted-by: Claude Opus 5 Signed-off-by: Mihirsinh Chavda <mihhhir08@gmail.com>
ab064f7 to
7ab9ab6
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7046 +/- ##
=======================================
Coverage 94.97% 94.97%
=======================================
Files 406 406
Lines 14269 14269
Branches 3272 3272
=======================================
Hits 13552 13552
Misses 717 717 🚀 New features to boost your workflow:
|
Found three relative links in the experimental docs that point at files that don't exist.
The experimental CHANGELOG links to
doc/upgrade-to-2.x.md, but the upgrade guide is at the repo root, so fromexperimental/it needs to be../doc/upgrade-to-2.x.md.The sdk-logs README links to
./src/config.ts, which isn't there anymore. The defaultLoggerConfignow lives in./src/config/LoggerConfigurators.tsso I pointed it there.The exporter-trace-otlp-grpc README links to
./test/certs/regenerate.sh, but that package has notest/certsdirectory. The script is inotlp-grpc-exporter-base, which that package already depends on.Docs only, no code changed, so I think this one needs the
Skip Changeloglabel.markdownlint-cli2passes on all three files.One thing I wasn't sure about, on the sdk-logs link you might prefer pointing at
src/internal/LoggerProviderSharedState.tssince that also defines aDEFAULT_LOGGER_CONFIG. Happy to switch it if that reads better.