-
-
Notifications
You must be signed in to change notification settings - Fork 148
Conversation
I drew a diagram that can easily understand the overall structure. Hope this diagram helps other contributors. |
Codecov Report
@@ Coverage Diff @@
## master #285 +/- ##
=======================================
Coverage 73.74% 73.74%
=======================================
Files 54 54
Lines 876 876
Branches 79 79
=======================================
Hits 646 646
Misses 195 195
Partials 35 35 Continue to review full report at Codecov.
|
It would be great to have it there too! Thanks for the contribution. |
@hyochan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rebase correctly. U are changing lots of files that are not in your purpose.
be18953
to
ab6f831
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Specify project
server
Description
Related Issues
x
Tests
x
Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.yarn lint && yarn tsc
yarn test
oryarn test -u
if you need to update snapshot.