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

Projects
None yet
6 participants
@QilongZhang
Copy link
Contributor

commented Jan 30, 2019

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 and others added some commits Dec 29, 2018

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

@codecov

This comment has been minimized.

Copy link

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.

@glmapper
Copy link

left a comment

+1

@straybirdzls straybirdzls merged commit b6b0201 into sofastack:master Jan 30, 2019

4 checks passed

clahub All contributors have signed the Contributor License Agreement.
Details
codecov/patch 67.5% of diff hit (target 62.7%)
Details
codecov/project 66.59% (+3.89%) compared to 959b30e
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.