Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Check placement exists before length check (#22060) (cherry-pick #22057) #22089

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

gcp-cherry-pick-bot[bot]
Copy link

Cherry-picked fix: Check placement exists before length check (#22060) (#22057)

Signed-off-by: Dale Haiducek 19750917+dhaiducek@users.noreply.github.com

Signed-off-by: Dale Haiducek <19750917+dhaiducek@users.noreply.github.com>
@gcp-cherry-pick-bot gcp-cherry-pick-bot bot requested a review from a team as a code owner February 28, 2025 15:55
Copy link
Member

@jannfis jannfis left a comment

Choose a reason for hiding this comment

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

LGTM

@jannfis jannfis merged commit 71fd4e5 into release-2.14 Feb 28, 2025
20 of 22 checks passed
@jannfis jannfis deleted the cherry-pick-32fb0d-release-2.14 branch February 28, 2025 16:41
Copy link

codecov bot commented Feb 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.18%. Comparing base (66db4b6) to head (71ce665).
Report is 2 commits behind head on release-2.14.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-2.14   #22089      +/-   ##
================================================
- Coverage         55.24%   55.18%   -0.07%     
================================================
  Files               338      338              
  Lines             57240    57240              
================================================
- Hits              31625    31588      -37     
- Misses            22920    22949      +29     
- Partials           2695     2703       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants