Skip to content

fix(storage): Do not move while creating current variable#15237

Merged
v-pratap merged 1 commit intogoogleapis:mainfrom
v-pratap:bug-move-options
Jun 30, 2025
Merged

fix(storage): Do not move while creating current variable#15237
v-pratap merged 1 commit intogoogleapis:mainfrom
v-pratap:bug-move-options

Conversation

@v-pratap
Copy link
Copy Markdown
Contributor

@v-pratap v-pratap commented Jun 26, 2025

The p.options is moved two times in this function, which is causing empty option at later stage.


This change is Reviewable

@v-pratap v-pratap requested review from a team June 26, 2025 09:59
@product-auto-label product-auto-label bot added the api: storage Issues related to the Cloud Storage API. label Jun 26, 2025
@v-pratap v-pratap requested a review from bajajneha27 June 26, 2025 10:09
@bajajneha27 bajajneha27 changed the title feat(storage): Do not move while creating current variable fix(storage): Do not move while creating current variable Jun 26, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.93%. Comparing base (05b46d0) to head (63a624c).
⚠️ Report is 49 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15237      +/-   ##
==========================================
- Coverage   92.93%   92.93%   -0.01%     
==========================================
  Files        2394     2394              
  Lines      215352   215352              
==========================================
- Hits       200132   200131       -1     
- Misses      15220    15221       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@scotthart scotthart left a comment

Choose a reason for hiding this comment

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

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @v-pratap)

@v-pratap v-pratap enabled auto-merge (squash) June 26, 2025 15:09
@v-pratap v-pratap merged commit 8727ff0 into googleapis:main Jun 30, 2025
79 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants