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

Refractor config_load.go #158

Merged
merged 7 commits into from May 28, 2021
Merged

Refractor config_load.go #158

merged 7 commits into from May 28, 2021

Conversation

lmhmhl
Copy link
Contributor

@lmhmhl lmhmhl commented Apr 22, 2021

  1. Bug fix, some key in yaml file with "_" can not convert into object, I solve this by adding mapstructre mapping in related model file.
  2. Complete the test of config_load.go.

@xiaoliu10 xiaoliu10 added this to the v0.3.0 milestone Apr 25, 2021
@codecov-commenter
Copy link

codecov-commenter commented Apr 25, 2021

Codecov Report

Merging #158 (7361635) into develop (c016f9e) will increase coverage by 0.19%.
The diff coverage is 53.70%.

❗ Current head 7361635 differs from pull request most recent head eaae075. Consider uploading reports for the commit eaae075 to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #158      +/-   ##
===========================================
+ Coverage    42.99%   43.19%   +0.19%     
===========================================
  Files           33       33              
  Lines         1742     1755      +13     
===========================================
+ Hits           749      758       +9     
+ Misses         890      888       -2     
- Partials       103      109       +6     
Impacted Files Coverage Δ
pkg/config/config_load.go 56.17% <53.70%> (+2.23%) ⬆️

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 c016f9e...eaae075. Read the comment docs.

@xiaoliu10 xiaoliu10 merged commit 51bb439 into apache:develop May 28, 2021
@xiaoliu10 xiaoliu10 modified the milestones: v0.3.0, v0.2.1 May 29, 2021
mark4z pushed a commit that referenced this pull request Nov 7, 2021
* Refractor config_load.go

* Refractor pixiu_start.go

* Refractor pixiu_start.go

* bug fix about read yml file. some key with _ can't read in object.

* Change DiscoverService to DiscoveryService.

Co-authored-by: zhangxun <18721825717@163.com>
Former-commit-id: 51bb439
tydhot pushed a commit to tydhot/dubbo-go-pixiu that referenced this pull request Nov 10, 2021
* Refractor config_load.go

* Refractor pixiu_start.go

* Refractor pixiu_start.go

* bug fix about read yml file. some key with _ can't read in object.

* Change DiscoverService to DiscoveryService.

Co-authored-by: zhangxun <18721825717@163.com>
Former-commit-id: b3940bb [formerly 51bb439]
Former-commit-id: 15c079b
bobtthp pushed a commit to bobtthp/dubbo-go-pixiu that referenced this pull request Dec 12, 2022
* Refractor config_load.go

* Refractor pixiu_start.go

* Refractor pixiu_start.go

* bug fix about read yml file. some key with _ can't read in object.

* Change DiscoverService to DiscoveryService.

Co-authored-by: zhangxun <18721825717@163.com>
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

4 participants