Skip to content

Conversation

@Vadman97
Copy link
Contributor

@Vadman97 Vadman97 commented Apr 22, 2025

Summary

Prepare npm SDK repository for publishing using LD npm token.

https://github.com/launchdarkly/terraform/pull/18409

How did you test this change?

CI, will manually trigger action once merged

Are there any deployment considerations?

no

Does this work require review from our design team?

no


Related Jira issue: OB-75: setup packaging for @launchdarkly o11y

@Vadman97 Vadman97 changed the title setup packaging for @launchdarkly/observability sdks [OB-75] setup packaging for @launchdarkly/observability sdks Apr 23, 2025
@Vadman97 Vadman97 marked this pull request as ready for review April 23, 2025 00:51
@Vadman97 Vadman97 requested a review from kinyoklion April 23, 2025 21:50
Copy link
Contributor

@ccschmitz-launchdarkly ccschmitz-launchdarkly left a comment

Choose a reason for hiding this comment

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

Makes sense to me, but will leave the final approval for @kinyoklion.

@Vadman97
Copy link
Contributor Author

Reviewed this offline with @kinyoklion - will merge and address follow ups as needed.

@Vadman97 Vadman97 merged commit 7b98759 into main Apr 24, 2025
16 checks passed
@Vadman97 Vadman97 deleted the vadim/ob-75 branch April 24, 2025 18:21
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.

3 participants