Skip to content

fix: Preserve Marketplace.Stubbed during client copy#4144

Open
alexandear wants to merge 1 commit intogoogle:masterfrom
alexandear-org:fix/marketplace-stub
Open

fix: Preserve Marketplace.Stubbed during client copy#4144
alexandear wants to merge 1 commit intogoogle:masterfrom
alexandear-org:fix/marketplace-stub

Conversation

@alexandear
Copy link
Copy Markdown
Contributor

This PR fixes a bug in initialize. initialize and copy always replace MarketplaceService with a fresh struct. This drops MarketplaceService.Stubbed if it was previously set to true.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.74%. Comparing base (5124fac) to head (bbcd02f).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4144   +/-   ##
=======================================
  Coverage   93.74%   93.74%           
=======================================
  Files         211      211           
  Lines       19685    19690    +5     
=======================================
+ Hits        18453    18458    +5     
  Misses       1034     1034           
  Partials      198      198           

☔ 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.

@gmlewis gmlewis changed the title fix: Preserve Marketplace.Stubbed during client copy fix: Preserve Marketplace.Stubbed during client copy Apr 10, 2026
Copy link
Copy Markdown
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @alexandear!
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.

cc: @stevehipwell - @zyfy29 - @Not-Dhananjay-Mishra - @munlicode

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NeedsReview PR is awaiting a review before merging.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants