Skip to content

Conversation

@vdiskg
Copy link
Contributor

@vdiskg vdiskg commented May 28, 2021

What's the purpose of this PR

fix docker-quick-start sql

Which issue(s) this PR fixes:

Fixes #3714

@codecov-commenter
Copy link

Codecov Report

Merging #3720 (4bd0498) into master (150955f) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3720      +/-   ##
============================================
- Coverage     50.37%   50.34%   -0.03%     
- Complexity     2373     2374       +1     
============================================
  Files           457      457              
  Lines         14435    14435              
  Branches       1457     1457              
============================================
- Hits           7271     7268       -3     
- Misses         6658     6661       +3     
  Partials        506      506              
Impacted Files Coverage Δ
...mework/apollo/portal/component/PortalSettings.java 57.81% <0.00%> (-7.82%) ⬇️
.../apollo/internals/RemoteConfigLongPollService.java 77.71% <0.00%> (+0.60%) ⬆️
...ework/apollo/internals/RemoteConfigRepository.java 89.37% <0.00%> (+0.62%) ⬆️

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 150955f...4bd0498. Read the comment docs.

Copy link
Member

@nobodyiam nobodyiam left a comment

Choose a reason for hiding this comment

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

LGTM

@nobodyiam nobodyiam merged commit 4e1a5c1 into apolloconfig:master May 28, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 28, 2021
@vdiskg vdiskg deleted the fix-oidc-sql branch June 27, 2021 08:58
@nobodyiam nobodyiam added this to the 1.9.0 milestone Jul 11, 2021
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.

docker部署sql错误

3 participants