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

[JS] Make instrumentation less verbose by removing the SemanticConventions enum and use the constants directly #117

Open
mikeldking opened this issue Jan 18, 2024 · 1 comment
Labels
language: js Related to JavaScript or Typescript integration

Comments

@mikeldking
Copy link
Contributor

Instead of writing SementicConventions.foo every time, we should just import the constants and use them directly so that the template strings are less verbose

Copy link

dosubot bot commented Jan 18, 2024

🤖

That sounds like a great idea! Making the instrumentation less verbose by using constants directly will definitely improve the readability of the code. If you have a proposed solution or are willing to create a pull request for this change, I encourage you to go ahead and contribute to the project. Your input will be valuable in making the code more maintainable and user-friendly.


This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.

Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.

If you want to continue the conversation, start your reply with @dosu-bot.

@dosubot dosubot bot added the language: js Related to JavaScript or Typescript integration label Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
language: js Related to JavaScript or Typescript integration
Projects
None yet
Development

No branches or pull requests

1 participant