Skip to content

Conversation

@lcian
Copy link
Member

@lcian lcian commented Jun 25, 2025

Hello 👋
As the tracing integration now supports sending logs, this PR enables them as we're doing in our other Rust-based services. Feel free to close this if you don't intend to use logs.

Changes:

  • Bump the SDK to the latest version
  • Disable the release-health feature flag, as that would add unnecessary dependencies
  • Send everything at or above INFO as logs instead of breadcrumbs (default for our tracing layer)

@lcian lcian requested a review from a team as a code owner June 25, 2025 07:32
@lcian lcian marked this pull request as draft June 26, 2025 08:16
@lcian lcian force-pushed the lcian/feat/add-logs branch from 3375c08 to 4c93d66 Compare June 27, 2025 07:54
@lcian lcian changed the base branch from main to lcian/bump-rust-toolchain June 27, 2025 07:54
@lcian lcian force-pushed the lcian/feat/add-logs branch 3 times, most recently from a5b2384 to 12a2f82 Compare June 27, 2025 11:26
evanpurkhiser pushed a commit that referenced this pull request Jun 30, 2025
Bumps the Rust version to 1.85 and fixes the new lints.
Needed for #324
Base automatically changed from lcian/bump-rust-toolchain to main June 30, 2025 20:37
@lcian lcian marked this pull request as ready for review July 1, 2025 08:23
@lcian lcian requested a review from evanpurkhiser July 14, 2025 09:05
@lcian lcian force-pushed the lcian/feat/add-logs branch from a9b529a to 7b545e9 Compare July 30, 2025 16:02
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