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

[Collector] Slight rewording of distributions and release info #6476

Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Mar 5, 2025

  • Reworks the Collector index page:
    • Splits "Status and releases" into separate sections
    • For releases, uses the title "Distributions and releases" and adds a link to the Distributions page
  • Contributes to [i18n] Factor prose out of shortcodes like docs/latest-release #6460 indirectly by eliminating the use of the shortcode in the Collector pages

Preview:

@chalin chalin requested a review from a team as a code owner March 5, 2025 21:39
@opentelemetrybot opentelemetrybot requested review from a team and codeboten and removed request for a team March 5, 2025 21:39
@chalin chalin force-pushed the chalin-im-rw-collector-releases-2025-03-05 branch from f4e2072 to af7961e Compare March 5, 2025 21:40
@opentelemetrybot opentelemetrybot requested a review from a team March 5, 2025 21:41
@chalin chalin force-pushed the chalin-im-rw-collector-releases-2025-03-05 branch from af7961e to a3973e1 Compare March 5, 2025 21:42
@chalin chalin requested a review from tiffany76 March 5, 2025 21:48
@chalin chalin added the sig-approval-missing Co-owning SIG didn't provide an approval label Mar 5, 2025
@chalin
Copy link
Contributor Author

chalin commented Mar 5, 2025

/fix:format

@opentelemetrybot
Copy link
Collaborator

You triggered fix:format action run at https://github.com/open-telemetry/opentelemetry.io/actions/runs/13687867195

@opentelemetrybot
Copy link
Collaborator

fix:format was successful.

IMPORTANT: (RE-)RUN /fix:all to ensure that there are no remaining check issues.

@chalin chalin force-pushed the chalin-im-rw-collector-releases-2025-03-05 branch from dd0a457 to 0975c56 Compare March 6, 2025 15:33
@chalin
Copy link
Contributor Author

chalin commented Mar 6, 2025

@codeboten and @open-telemetry/collector-approvers - friendly ping. I'd like to get this merged so that progress can be made on followup changes (to other sections).

@chalin
Copy link
Contributor Author

chalin commented Mar 6, 2025

Thanks @mx-psi (for @open-telemetry/collector-approvers), I've committed your suggesting. Merging.

chalin and others added 4 commits March 6, 2025 11:47
Co-authored-by: Tiffany Hrabusa <30397949+tiffany76@users.noreply.github.com>
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
@chalin chalin force-pushed the chalin-im-rw-collector-releases-2025-03-05 branch from 72e51e7 to 925dbf6 Compare March 6, 2025 16:47
@chalin chalin removed the request for review from codeboten March 6, 2025 16:48
@opentelemetrybot opentelemetrybot requested review from a team and jpkrohling and removed request for a team March 6, 2025 16:48
@chalin chalin merged commit 2f99eb6 into open-telemetry:main Mar 6, 2025
18 checks passed
@chalin chalin deleted the chalin-im-rw-collector-releases-2025-03-05 branch March 6, 2025 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
copyedit sig:collector sig-approval-missing Co-owning SIG didn't provide an approval
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants