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

fix: update ADOT links and add missing languages #6585

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

garysassano
Copy link
Member

@garysassano garysassano commented Mar 21, 2025

This PR updates the AWS Distro for OpenTelemetry (ADOT) links and adds missing language entries to the distributions list.

Changes

  • Updated existing ADOT entries to point to the latest active repositories referenced in the official AWS docs
  • Added additional ADOT entries for missing languages

Preview: https://deploy-preview-6585--opentelemetry.netlify.app/ecosystem/distributions/

@garysassano garysassano requested a review from a team as a code owner March 21, 2025 05:52
@opentelemetrybot opentelemetrybot requested review from a team March 21, 2025 05:53
@svrnm
Copy link
Member

svrnm commented Mar 21, 2025

thanks for your contribution @garysassano! Your PR touches files unrelated to your suggested change, can you clean up your PR and update accordingly? Thanks

pnpm-lock.yaml Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this file.

@opentelemetrybot opentelemetrybot requested review from a team March 21, 2025 12:19
@garysassano
Copy link
Member Author

thanks for your contribution @garysassano! Your PR touches files unrelated to your suggested change, can you clean up your PR and update accordingly? Thanks

Those files weren't touched by me, but by the log:test-and-fix script I run to update the refcache.

@chalin
Copy link
Contributor

chalin commented Mar 21, 2025

@garysassano - that's what I suspected. Let me fix main and get back to you.

@chalin
Copy link
Contributor

chalin commented Mar 21, 2025

Issue created to address the problem that surfaced here:

@chalin
Copy link
Contributor

chalin commented Mar 21, 2025

@garysassano - After the following PR is merged:

We'll rebase and that should cleanup your PR.

@chalin
Copy link
Contributor

chalin commented Mar 21, 2025

You still need to remove pnpm-lock.yaml. I've added it to .gitignores in the previously mentioned PR.

Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

LGTM from a CI/infra and cleanup perspective.

@svrnm or another @open-telemetry/docs-approvers, will let you give a final approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants