Skip to content

feat(multiplexer): use slash to separate rpc id#3583

Merged
bytemain merged 1 commit intomainfrom
feat/use-slash-to-sep-rpc-id
Apr 24, 2024
Merged

feat(multiplexer): use slash to separate rpc id#3583
bytemain merged 1 commit intomainfrom
feat/use-slash-to-sep-rpc-id

Conversation

@bytemain
Copy link
Copy Markdown
Member

Types

  • 🎉 New Features

Background or solution

看起来 /|| 要好看一点

Changelog

@railway-app
Copy link
Copy Markdown

railway-app Bot commented Apr 23, 2024

This PR is being deployed to Railway 🚅

core: ◻️ REMOVED

@bytemain bytemain changed the title feat: use slash to separate rpc id feat(mutliplexer): use slash to separate rpc id Apr 23, 2024
@opensumi opensumi Bot added the 🎨 feature feature required label Apr 23, 2024
@bytemain bytemain changed the title feat(mutliplexer): use slash to separate rpc id feat(multiplexer): use slash to separate rpc id Apr 23, 2024
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.00%. Comparing base (5d54570) to head (ef635de).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3583      +/-   ##
==========================================
- Coverage   56.01%   56.00%   -0.02%     
==========================================
  Files        1490     1490              
  Lines       91262    91265       +3     
  Branches    18716    18716              
==========================================
- Hits        51124    51116       -8     
- Misses      33279    33291      +12     
+ Partials     6859     6858       -1     
Flag Coverage Δ
jsdom 51.40% <100.00%> (+<0.01%) ⬆️
node 15.63% <90.47%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Comment thread packages/connection/src/common/rpc/multiplexer.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 feature feature required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants