Skip to content

refactor(core): select discovery backend from configured type - #719

Merged
dsx137 merged 2 commits into
openimsdk:mainfrom
dsx137:main
Jul 22, 2026
Merged

refactor(core): select discovery backend from configured type#719
dsx137 merged 2 commits into
openimsdk:mainfrom
dsx137:main

Conversation

@dsx137

@dsx137 dsx137 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

🅰 Please add the issue ID after "Fixes #"

Fixes #

@pull-request-size pull-request-size Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jul 21, 2026
@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@fe678ab). Learn more about missing BASE report.

Files with missing lines Patch % Lines
pkg/common/kdisc/discoveryregister.go 0.00% 16 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##             main    #719   +/-   ##
======================================
  Coverage        ?   0.00%           
======================================
  Files           ?     130           
  Lines           ?   13167           
  Branches        ?       0           
======================================
  Hits            ?       0           
  Misses          ?   13167           
  Partials        ?       0           

☔ View full report in Codecov by Harness.
📢 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.

@dsx137
dsx137 added this pull request to the merge queue Jul 22, 2026
Merged via the queue into openimsdk:main with commit 73e7b82 Jul 22, 2026
8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants