Skip to content

Comments

add missing Capability#6487

Closed
asukaminato0721 wants to merge 3 commits intoapache:mainfrom
asukaminato0721:fix-cpp-test
Closed

add missing Capability#6487
asukaminato0721 wants to merge 3 commits intoapache:mainfrom
asukaminato0721:fix-cpp-test

Conversation

@asukaminato0721
Copy link
Contributor

Which issue does this PR close?

Closes #6296 (comment) .

Rationale for this change

add check for cap.

What changes are included in this PR?

Are there any user-facing changes?

@asukaminato0721 asukaminato0721 requested a review from Xuanwo as a code owner August 5, 2025 08:55
@asukaminato0721 asukaminato0721 marked this pull request as draft August 5, 2025 08:55
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Aug 5, 2025
@asukaminato0721 asukaminato0721 marked this pull request as ready for review August 5, 2025 09:12
@Xuanwo
Copy link
Member

Xuanwo commented Aug 5, 2025

Hi, please submit PR from apache upstream, otherwise we can't correctly test it.

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

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants