Skip to content

Comments

chore(services/obs): Use QueryPairsWriter for url write#5985

Merged
Xuanwo merged 1 commit intoapache:mainfrom
asukaminato0721:obs
Apr 9, 2025
Merged

chore(services/obs): Use QueryPairsWriter for url write#5985
Xuanwo merged 1 commit intoapache:mainfrom
asukaminato0721:obs

Conversation

@asukaminato0721
Copy link
Contributor

Which issue does this PR close?

part of #5976.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@asukaminato0721 asukaminato0721 requested a review from Xuanwo as a code owner April 8, 2025 07:27
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. core releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell services/obs labels Apr 8, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Nice change, thank you @asukaminato0721!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 9, 2025
@Xuanwo Xuanwo merged commit 737f55e into apache:main Apr 9, 2025
91 checks passed
@asukaminato0721 asukaminato0721 deleted the obs branch April 16, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core lgtm This PR has been approved by a maintainer releases-note/chore The PR has a title that begins with "chore" or changes other small things that hard to tell services/obs size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants