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

Warning messaging #145

Open
G4Vi opened this issue Nov 21, 2023 · 0 comments
Open

Warning messaging #145

G4Vi opened this issue Nov 21, 2023 · 0 comments
Assignees

Comments

@G4Vi
Copy link
Contributor

G4Vi commented Nov 21, 2023

Working on Updated Versioning. I added deprecation warnings for the old namespace dylibso_observe. However, I'm not sure they are going the right place or actually making it to the observe-sdk user across the languages and adapters. We should decide on a consistently policy for this and apply it everywhere.

Currently, the rust sdk outputs with log::warn! and the basic example is setup to by default log with warnings.

The go-sdk outputs to the standard logger with log.Println.

The js-sdk outputs with console.warn.

@neuronicnobody neuronicnobody added the question Further information is requested label Nov 27, 2023
@neuronicnobody neuronicnobody removed the question Further information is requested label Dec 11, 2023
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

No branches or pull requests

2 participants