search

ali-sdk / ali-ons

Aliyun Open Notification Service Client
MIT License
151 stars 43 forks source link

feat: support set unitName #101

Closed gxcsoccer closed 3 years ago

gxcsoccer commented 3 years ago

image

需要支持单元化

codecov[bot] commented 3 years ago

Codecov Report

Merging #101 (56ee715) into master (4d512fb) will increase coverage by 91.66%. The diff coverage is 83.33%.

:exclamation: Current head 56ee715 differs from pull request most recent head ef3103f. Consider uploading reports for the commit ef3103f to get more accurate results Impacted file tree graph

@@             Coverage Diff             @@
##           master     #101       +/-   ##
===========================================
+ Coverage        0   91.66%   +91.66%     
===========================================
  Files           0       35       +35     
  Lines           0     2148     +2148     
  Branches        0      354      +354     
===========================================
+ Hits            0     1969     +1969     
- Misses          0      179      +179
Impacted Files Coverage Δ
lib/client_config.js 97.05% <80.00%> (ø)
lib/remoting_client.js 94.94% <83.33%> (ø)
lib/consumer/mq_push_consumer.js 93.30% <100.00%> (ø)
lib/protocol/consume_type.js 100.00% <0.00%> (ø)
lib/mq_client.js 89.87% <0.00%> (ø)
...sumer/rebalance/allocate_message_queue_strategy.js 20.00% <0.00%> (ø)
lib/store/remote_broker.js 96.47% <0.00%> (ø)
lib/message_queue.js 100.00% <0.00%> (ø)
lib/message/message_const.js 100.00% <0.00%> (ø)
lib/process_queue.js 77.41% <0.00%> (ø)
... and 28 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 4d512fb...ef3103f. Read the comment docs.

denghongcai commented 3 years ago

LGTM

denghongcai commented 3 years ago