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

sqlite3 instrumentation support added #255

Closed

Conversation

deepakdinesh1123
Copy link
Contributor

Please let me know if this is something that you guys would like to add to logfire, I will add some more tests and make any changes that you guys suggest.

TODO

  • Add more tests

Copy link

codecov bot commented Jun 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@deepakdinesh1123 deepakdinesh1123 marked this pull request as draft June 9, 2024 14:33
@alexmojaki
Copy link
Contributor

Yes this would be appreciated. It'd also need docs.

@Kludex
Copy link
Member

Kludex commented Jul 17, 2024

@deepakdinesh1123 Are you willing to resume your work on this PR?

@deepakdinesh1123
Copy link
Contributor Author

@Kludex No, I am busy at work so I have not been able to work on this.

@Kludex
Copy link
Member

Kludex commented Jul 29, 2024

@deepakdinesh1123 I'll close this PR since you don't have time to finish.

Let us know if you want to do it later. We'll be happy to help. :)

@Kludex Kludex closed this Jul 29, 2024
@Kludex
Copy link
Member

Kludex commented Jul 29, 2024

We'll eventually implement this anyway.

@alexmojaki
Copy link
Contributor

This is being done in #634

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