-
Notifications
You must be signed in to change notification settings - Fork 158
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
feat: tracing added for imply and open telemetry #1424
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
How to use the Graphite Merge QueueAdd the label graphite-merge-queue to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. |
…yplatform/aa-sdk into jade/wip/tracing-sdk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks great, there might be one case to check here though. So we initially added the headers on all clients. However, we ended up having to do it just on alchemy clients because adding headers was causing issues for someone who was using aa-sdk core to call a different RPC provider. that RPC provider was throwing errors due to unknown headers or extra headers being added
f8eb05e
to
09c8352
Compare
09c8352
to
66acd19
Compare
a2f4de3
to
269c8db
Compare
269c8db
to
955646d
Compare
955646d
to
2877b9f
Compare
Pull Request Checklist
yarn test
)site
folder, and guidelines for updating/adding docs can be found in the contribution guide)feat!: breaking change
)yarn lint:check
) and fix any issues? (yarn lint:write
)PR-Codex overview
This PR primarily focuses on updating the documentation and code to improve consistency in parameter naming, specifically changing instances of
client
toclient_
. It also introduces new tracing functionalities and updates client handling in various methods.Detailed summary
client
toclient_
in multiple.mdx
files and TypeScript files.TraceHeader
class and related methods.TraceHeader
methods.