Skip to content

chore: remove kramdown#2097

Merged
kaylareopelle merged 1 commit into
open-telemetry:mainfrom
thompson-tomo:remove_kramdown
May 8, 2026
Merged

chore: remove kramdown#2097
kaylareopelle merged 1 commit into
open-telemetry:mainfrom
thompson-tomo:remove_kramdown

Conversation

@thompson-tomo
Copy link
Copy Markdown
Contributor

Remove kramdown from sem conv gem which appears not to be used.

Also The docs appear to not be published as link points to a url which lacks a link to sem-conv.

Copy link
Copy Markdown
Contributor

@kaylareopelle kaylareopelle left a comment

Choose a reason for hiding this comment

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

The kramdown removal looks good to me.

Also The docs appear to not be published as link points to a url which lacks a link to sem-conv.

I can find the docs on rubydoc: https://rubydoc.info/gems/opentelemetry-semantic_conventions

Should the link that you're referring to be somewhere on this page? https://open-telemetry.github.io/opentelemetry-ruby/

@thompson-tomo
Copy link
Copy Markdown
Contributor Author

@kaylareopelle according to the gemspec the documentation is at https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-semantic_conventions/v#{OpenTelemetry::SemanticConventions::VERSION which redirects to https://open-telemetry.github.io/opentelemetry-ruby/ and semantic conventions is not listed.

Let's update the links seperately.

@kaylareopelle kaylareopelle merged commit 52ae994 into open-telemetry:main May 8, 2026
63 checks passed
@thompson-tomo thompson-tomo deleted the remove_kramdown branch May 9, 2026 00:43
@thompson-tomo
Copy link
Copy Markdown
Contributor Author

Links have been addressed in #2123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants