Skip to content

Harmonize ObservationConvention parameters in gRPC support#49787

Merged
snicoll merged 2 commits intospring-projects:mainfrom
therepanic:polish-GrpcClientObservationAutoConfiguration
Mar 26, 2026
Merged

Harmonize ObservationConvention parameters in gRPC support#49787
snicoll merged 2 commits intospring-projects:mainfrom
therepanic:polish-GrpcClientObservationAutoConfiguration

Conversation

@therepanic
Copy link
Copy Markdown
Contributor

Let's rename GrpcClientObservationConvention to convention to maintain style and consistency with GrpcServerObservationAutoConfiguration

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 25, 2026
@philwebb philwebb changed the title Rename parameter GrpcClientObservationConvention to convention Rename parameter 'GrpcClientObservationConvention' to 'convention' Mar 25, 2026
@philwebb philwebb added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 25, 2026
@philwebb philwebb added this to the 4.1.x milestone Mar 25, 2026
@snicoll
Copy link
Copy Markdown
Member

snicoll commented Mar 26, 2026

Woops that name is odd, good catch @therepanic!

@snicoll snicoll self-assigned this Mar 26, 2026
@snicoll snicoll modified the milestones: 4.1.x, 4.1.0-M4 Mar 26, 2026
See spring-projectsgh-49787

Signed-off-by: Andrey Litvitski <andrey1010102008@gmail.com>
@snicoll snicoll changed the title Rename parameter 'GrpcClientObservationConvention' to 'convention' Harmonize ObservationConvention parameters in gRPC support Mar 26, 2026
Such parameters are usually named "customConvention" so this commit
aligns to that naming.

See spring-projectsgh-49730
@snicoll snicoll force-pushed the polish-GrpcClientObservationAutoConfiguration branch from 596f886 to d7a4561 Compare March 26, 2026 07:41
@snicoll snicoll merged commit 910340c into spring-projects:main Mar 26, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants