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

Cherry pick PR from 2.6.0 #355

Merged
merged 21 commits into from Jan 30, 2019

Conversation

QilongZhang
Copy link
Contributor

cherry-pick all PR except #325 from milestone 2.6.0.

Endpoint is refactored in Spring Boot 2.x, i guess you need another PR for #313

@QilongZhang QilongZhang added this to the 3.1.2 milestone Jan 30, 2019
@codecov
Copy link

codecov bot commented Jan 30, 2019

Codecov Report

Merging #355 into master will increase coverage by 3.89%.
The diff coverage is 67.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #355      +/-   ##
==========================================
+ Coverage    62.7%   66.59%   +3.89%     
==========================================
  Files          76      130      +54     
  Lines        1984     3739    +1755     
  Branches      236      511     +275     
==========================================
+ Hits         1244     2490    +1246     
- Misses        581      958     +377     
- Partials      159      291     +132
Impacted Files Coverage Δ
...y/sofa/infra/constants/SofaBootInfraConstants.java 0% <ø> (ø) ⬆️
...ing/config/SofaRuntimeConfigurationProperties.java 100% <ø> (ø) ⬆️
...ofa/runtime/spring/factory/ServiceFactoryBean.java 66.66% <ø> (ø) ⬆️
...ntime/spring/SofaRuntimeContextAwareProcessor.java 87.5% <0%> (+16.07%) ⬆️
...va/com/alipay/sofa/runtime/spi/log/SofaLogger.java 57.89% <0%> (-6.82%) ⬇️
...runtime/spring/ClientFactoryBeanPostProcessor.java 83.33% <0%> (-4.17%) ⬇️
...e/spring/ReferenceAnnotationBeanPostProcessor.java 66.17% <0%> (-1.48%) ⬇️
...a/runtime/component/impl/ComponentManagerImpl.java 61.22% <0%> (-6.6%) ⬇️
.../sofa/runtime/api/client/param/ExtensionParam.java 100% <100%> (ø)
.../sofa/runtime/spi/component/AbstractComponent.java 62.96% <100%> (+17.31%) ⬆️
... and 128 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 959b30e...2438ed2. Read the comment docs.

Copy link
Contributor

@glmapper glmapper left a comment

Choose a reason for hiding this comment

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

+1

@straybirdzls straybirdzls merged commit b6b0201 into sofastack:master Jan 30, 2019
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

6 participants