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

add plugin for base #116

Merged
merged 7 commits into from
Sep 26, 2023
Merged

add plugin for base #116

merged 7 commits into from
Sep 26, 2023

Conversation

lvjing2
Copy link
Collaborator

@lvjing2 lvjing2 commented Sep 21, 2023

  1. 更改 arklet 目录到 sofa-serverless-runtime 目录
  2. 增加模块引入的 starter,基座和模块公共 bundle
  3. 增加基座 springboot 插件,记录 SpringContext,允许模块动态卸载

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Attention: 37 lines in your changes are missing coverage. Please review.

Comparison is base (fc07fc0) 19.20% compared to head (1d0a821) 22.33%.
Report is 46 commits behind head on master.

❗ Current head 1d0a821 differs from pull request most recent head 24e2dde. Consider uploading reports for the commit 24e2dde to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #116      +/-   ##
============================================
+ Coverage     19.20%   22.33%   +3.13%     
- Complexity       78      102      +24     
============================================
  Files            45       65      +20     
  Lines          1156     1379     +223     
  Branches         56       72      +16     
============================================
+ Hits            222      308      +86     
- Misses          922     1049     +127     
- Partials         12       22      +10     
Files Coverage Δ
...erverless/arklet/core/ArkletComponentRegistry.java 95.12% <ø> (ø)
...pay/sofa/serverless/arklet/core/api/ApiClient.java 100.00% <ø> (ø)
...ofa/serverless/arklet/core/api/model/Response.java 0.00% <ø> (ø)
...serverless/arklet/core/api/model/ResponseCode.java 0.00% <ø> (ø)
...erless/arklet/core/api/tunnel/http/HttpTunnel.java 56.75% <ø> (ø)
...et/core/api/tunnel/http/netty/NettyHttpServer.java 28.98% <ø> (ø)
.../core/api/tunnel/http/netty/RequestValidation.java 0.00% <ø> (ø)
...erless/arklet/core/command/CommandServiceImpl.java 31.03% <ø> (ø)
...ss/arklet/core/command/builtin/BuiltinCommand.java 100.00% <ø> (ø)
...et/core/command/builtin/handler/HealthHandler.java 7.14% <ø> (ø)
... and 55 more

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

Copy link
Collaborator

@TomorJM TomorJM left a comment

Choose a reason for hiding this comment

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

LGTM

@lvjing2 lvjing2 force-pushed the youji-dev branch 2 times, most recently from 50357e0 to 21c047d Compare September 26, 2023 09:15
Copy link
Collaborator

@yuanyuancin yuanyuancin left a comment

Choose a reason for hiding this comment

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

LGFM

@yuanyuancin yuanyuancin merged commit 99a7238 into master Sep 26, 2023
3 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