Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

fix local registry #115

Merged
merged 3 commits into from
Nov 12, 2018
Merged

fix local registry #115

merged 3 commits into from
Nov 12, 2018

Conversation

leizhiyuan
Copy link
Contributor

Motivation:

fix when there are some settings in local registry for extends

Modification:

Describe the idea and modifications you've done.

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

@codecov
Copy link

codecov bot commented Nov 12, 2018

Codecov Report

Merging #115 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #115   +/-   ##
=========================================
  Coverage     54.85%   54.85%           
  Complexity      393      393           
=========================================
  Files            53       53           
  Lines          1741     1741           
  Branches        310      310           
=========================================
  Hits            955      955           
  Misses          606      606           
  Partials        180      180
Impacted Files Coverage Δ Complexity Δ
...ay/sofa/rpc/boot/config/LocalFileConfigurator.java 78.57% <100%> (ø) 5 <1> (ø) ⬇️
...fa/rpc/boot/container/RegistryConfigContainer.java 46.55% <100%> (ø) 9 <0> (ø) ⬇️

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 feb9084...498a087. Read the comment docs.

@JervyShi JervyShi merged commit 6ababd5 into sofastack:master Nov 12, 2018
@leizhiyuan leizhiyuan deleted the fix_local_registry branch November 26, 2018 02:58
@leizhiyuan leizhiyuan added this to the 6.0.1 milestone Nov 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants