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.

Sign up for GitHub

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

Jump to bottom

feat: support Consul as ServiceRegistry #666

Merged
merged 50 commits into from
Aug 8, 2024

write test codes in one style

49c5909
Select commit
Loading
Failed to load commit list.
Merged

feat: support Consul as ServiceRegistry #666

write test codes in one style
49c5909
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Aug 7, 2024 in 0s

76.78% of diff hit (target 70.00%)

View this Pull Request on Codecov

76.78% of diff hit (target 70.00%)

Annotations

Check warning on line 138 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L138 

Added line #L138 was not covered by tests

Check warning on line 144 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L140-L144 

Added lines #L140 - L144 were not covered by tests

Check warning on line 146 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L146 

Added line #L146 was not covered by tests

Check warning on line 148 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L148 

Added line #L148 was not covered by tests

Check warning on line 218 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L217-L218 

Added lines #L217 - L218 were not covered by tests

Check warning on line 231 in controller/registries/consul/config.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controller/registries/consul/config.go#L231 

Added line #L231 was not covered by tests
View more details on Codecov