Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IA] More shortcodes for exporter pages #3931

Merged
merged 5 commits into from
Feb 7, 2024

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Feb 2, 2024

Related to #3859, this PR heavily applies shortcodes to deduplicate content across exporter pages (currently java, python, JS)

@open-telemetry/docs-approvers PTAL!

@cartermp
Copy link
Contributor

cartermp commented Feb 2, 2024

Is there a reason why this is a draft?

@svrnm
Copy link
Member Author

svrnm commented Feb 2, 2024

Is there a reason why this is a draft?

I didn't want to trigger all the maintainers yet. Get your (docs) feedback first

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

Overall I am a fan. This should also make normalizing exporter docs for other languages easier.

@svrnm svrnm marked this pull request as ready for review February 5, 2024 09:09
@svrnm svrnm requested review from a team as code owners February 5, 2024 09:09
@svrnm svrnm merged commit fb40e74 into open-telemetry:main Feb 7, 2024
14 checks passed
@svrnm svrnm deleted the more-shortcodes-for-exporter-pages branch February 7, 2024 08:34

{{/*
below list needs to grow until all languages are updated to a consistent structure.
*/}} {{ if in (slice "python" "js" "java") $lang -}}
Copy link
Contributor

Choose a reason for hiding this comment

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

@svrnm - instead of {{ if in (slice "python" "js" "java") $lang -}} should a field be added to the data file?

Copy link
Contributor

Choose a reason for hiding this comment

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

@chalin chalin added the IA Information architecture rework label Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IA Information architecture rework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants