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: move semantic-conventions to regular dependencies #310

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

dobesv
Copy link
Contributor

@dobesv dobesv commented Jan 20, 2021

These dependencies are imported at runtime but were only declared as
devDependencies. This causes problems for people using these
packages with yarn pnp.

These dependencies are imported at runtime but were only declared as
devDependencies.  This causes problems for people using these
packages with yarn pnp.
@dobesv dobesv requested a review from a team January 20, 2021 18:32
@codecov
Copy link

codecov bot commented Jan 20, 2021

Codecov Report

Merging #310 (0789ae9) into master (f8d8686) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #310   +/-   ##
=======================================
  Coverage   95.41%   95.41%           
=======================================
  Files         115      115           
  Lines        6087     6087           
  Branches      592      592           
=======================================
  Hits         5808     5808           
  Misses        279      279           

@obecny obecny merged commit 6fab815 into open-telemetry:master Jan 20, 2021
@obecny obecny added the bug Something isn't working label Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants