Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

issue: 226 Expose CP for external HomeStore client #227

Merged
merged 3 commits into from
Nov 15, 2023
Merged

issue: 226 Expose CP for external HomeStore client #227

merged 3 commits into from
Nov 15, 2023

Conversation

yamingk
Copy link
Contributor

@yamingk yamingk commented Nov 14, 2023

This PR is needed by HomeObject, Issue 109: eBay/HomeObject#109 to allow HomeObject to participant cp.

The testing is going to happen with HomeObject issue-109

Locally tested HomeObj build with private homestore package which passed the HOmeObj build test.

@yamingk yamingk requested a review from hkadayam November 14, 2023 00:43
@yamingk yamingk linked an issue Nov 14, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (02b16cc) 70.07% compared to head (8d07b90) 70.55%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
+ Coverage   70.07%   70.55%   +0.48%     
==========================================
  Files          97       97              
  Lines        7932     7929       -3     
  Branches     1018     1019       +1     
==========================================
+ Hits         5558     5594      +36     
+ Misses       1881     1850      -31     
+ Partials      493      485       -8     

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

@yamingk yamingk merged commit 1fd188a into eBay:master Nov 15, 2023
16 checks passed
@yamingk yamingk deleted the yk_cp branch December 6, 2023 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose CP for external HomeStore Clients
3 participants