Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

代码精简 #5824

Closed
wants to merge 3 commits into from
Closed

代码精简 #5824

wants to merge 3 commits into from

Conversation

mek1986
Copy link
Contributor

@mek1986 mek1986 commented Apr 3, 2024

  1. 代码逻辑简化
  2. 无用代码消除

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 46.15385% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 58.68%. Comparing base (bb6077d) to head (73a95ed).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #5824      +/-   ##
============================================
- Coverage     58.68%   58.68%   -0.01%     
- Complexity    27927    27928       +1     
============================================
  Files          1341     1341              
  Lines        128288   128275      -13     
  Branches      24209    24207       -2     
============================================
- Hits          75287    75279       -8     
+ Misses        43614    43611       -3     
+ Partials       9387     9385       -2     
Files Coverage Δ
...n/java/com/alibaba/druid/pool/DruidDataSource.java 70.47% <46.15%> (-0.09%) ⬇️

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@lizongbo
Copy link
Collaborator

lizongbo commented May 4, 2024

代码存在冲突,请解决冲突后更新pr

@mek1986
Copy link
Contributor Author

mek1986 commented May 4, 2024

代码存在冲突,请解决冲突后更新pr

收到

@mek1986 mek1986 closed this May 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants