Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

symmetric deploy #401

Merged
merged 7 commits into from
Jan 2, 2024
Merged

symmetric deploy #401

merged 7 commits into from
Jan 2, 2024

Conversation

zemingzhu
Copy link
Collaborator

Motivation

Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.

Modification

Describe the idea and modifications you've done.

Result

Resolved or fixed #.

If there is no issue then describe the changes introduced by this PR.

Copy link

sofastack-cla bot commented Dec 13, 2023

Hi @zemingzhu, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

Copy link

codecov bot commented Dec 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b4db721) 81.67% compared to head (6a298b1) 76.31%.
Report is 11 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #401      +/-   ##
============================================
- Coverage     81.67%   76.31%   -5.37%     
============================================
  Files            41       17      -24     
  Lines          1015     1617     +602     
  Branches         90        0      -90     
============================================
+ Hits            829     1234     +405     
- Misses          135      278     +143     
- Partials         51      105      +54     

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

@liu-657667
Copy link
Collaborator

LGTM

Copy link
Collaborator

@gold300jin gold300jin left a comment

Choose a reason for hiding this comment

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

LGTM

@gold300jin gold300jin merged commit b82a9f2 into master Jan 2, 2024
9 of 10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants