Skip to content

docs: updated the "Read The Docs" documentation#4728

Merged
xrmx merged 10 commits intoopen-telemetry:mainfrom
jscancella:main
Aug 22, 2025
Merged

docs: updated the "Read The Docs" documentation#4728
xrmx merged 10 commits intoopen-telemetry:mainfrom
jscancella:main

Conversation

@jscancella
Copy link
Copy Markdown
Contributor

Description

I noticed that several of the examples didn't link to their source code on github even though others did. To be consistent I added a link to the example in the README.rst files.

While working on this I also noticed that there was an example for prometheus but no entry in the Read The Docs generated, so I added one following the same style as the other pages.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Documentation

How Has This Been Tested?

generated documentation locally and inspected the html files. I checked all the links and make sure they were pointing to correct github example page.

@jscancella jscancella requested a review from a team as a code owner August 18, 2025 23:28
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Aug 18, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@xrmx
Copy link
Copy Markdown
Contributor

xrmx commented Aug 19, 2025

@jscancella Thanks for the PR but you need to sign the CLA in order to contribute to OpenTelemetry.

@xrmx xrmx moved this to Reviewed PR that needs fixing in Python PR digest Aug 19, 2025
@jscancella
Copy link
Copy Markdown
Contributor Author

ok, I just signed the agreement. Will it automatically pick that up or is there something else I need to do to kick it off again?

Comment thread docs/examples/metrics/prometheus-grafana/README.rst Outdated
Comment thread docs/examples/metrics/prometheus-grafana/README.rst Outdated
@tammy-baylis-swi
Copy link
Copy Markdown
Contributor

Please could you also add an entry to the Changelog for this update.

Comment thread docs/examples/metrics/prometheus-grafana/README.rst Outdated
@jscancella
Copy link
Copy Markdown
Contributor Author

Please could you also add an entry to the Changelog for this update.

Added an entry - but let me know if there is a different format or anything that needs to be changed

Comment thread CHANGELOG.md
@tammy-baylis-swi tammy-baylis-swi moved this from Reviewed PR that needs fixing to Easy to review / merge / close in Python PR digest Aug 21, 2025
Copy link
Copy Markdown
Contributor

@tammy-baylis-swi tammy-baylis-swi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, thank you!

Comment thread CHANGELOG.md Outdated
@xrmx xrmx enabled auto-merge (squash) August 22, 2025 14:30
@xrmx xrmx merged commit 05343a5 into open-telemetry:main Aug 22, 2025
371 checks passed
@github-project-automation github-project-automation Bot moved this from Easy to review / merge / close to Done in Python PR digest Aug 22, 2025
JWinermaSplunk pushed a commit to JWinermaSplunk/opentelemetry-python that referenced this pull request Feb 17, 2026
* docs: updated the ReadTheDocs documentation to all have links to the examples for consistency

* Update docs/examples/metrics/prometheus-grafana/README.rst

Co-authored-by: Tammy Baylis <[email protected]>

* Added changelog entry as requested in PR feedback

* fixed link to localhost to see the metrics

* just use a plain http link

* Update CHANGELOG.md

Co-authored-by: Riccardo Magliocchetti <[email protected]>

* fix tox reported issues

* fix capitalization of Prometheus

* Update CHANGELOG.md

---------

Co-authored-by: Tammy Baylis <[email protected]>
Co-authored-by: Riccardo Magliocchetti <[email protected]>
JWinermaSplunk pushed a commit to JWinermaSplunk/opentelemetry-python that referenced this pull request Feb 17, 2026
* docs: updated the ReadTheDocs documentation to all have links to the examples for consistency

* Update docs/examples/metrics/prometheus-grafana/README.rst

Co-authored-by: Tammy Baylis <[email protected]>

* Added changelog entry as requested in PR feedback

* fixed link to localhost to see the metrics

* just use a plain http link

* Update CHANGELOG.md

Co-authored-by: Riccardo Magliocchetti <[email protected]>

* fix tox reported issues

* fix capitalization of Prometheus

* Update CHANGELOG.md

---------

Co-authored-by: Tammy Baylis <[email protected]>
Co-authored-by: Riccardo Magliocchetti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants