Skip to content

Conversation

@matt-hensley
Copy link
Contributor

@matt-hensley matt-hensley commented Jan 15, 2025

Changes

Adopt v1.23 database span network attributes. This does not update any other attributes that may be out of date.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

@github-actions github-actions bot added the comp:instrumentation.stackexchangeredis Things related to OpenTelemetry.Instrumentation.StackExchangeRedis label Jan 15, 2025
@matt-hensley matt-hensley marked this pull request as ready for review January 17, 2025 14:17
@matt-hensley matt-hensley requested a review from a team as a code owner January 17, 2025 14:17
# Conflicts:
#	src/OpenTelemetry.Instrumentation.StackExchangeRedis/CHANGELOG.md
Copy link
Member

@Kielek Kielek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, only changelog formating.

@Kielek Kielek merged commit 23e2123 into open-telemetry:main Jan 22, 2025
213 of 214 checks passed
@codecov
Copy link

codecov bot commented Jan 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (71655ce) to head (d0ea539).
Report is 685 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #2468       +/-   ##
==========================================
- Coverage   73.91%       0   -73.92%     
==========================================
  Files         267       0      -267     
  Lines        9615       0     -9615     
==========================================
- Hits         7107       0     -7107     
+ Misses       2508       0     -2508     

see 263 files with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:instrumentation.stackexchangeredis Things related to OpenTelemetry.Instrumentation.StackExchangeRedis

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants