quarkiverse / quarkus-dapr

The Distributed Application Runtime (Dapr) provides APIs that simplify microservice connectivity

https://dapr.io/

Apache License 2.0

30 stars 16 forks source link

Add DevServices #139

Closed mcruzdev closed 8 months ago

mcruzdev commented 8 months ago

This pull request aims to fix the following issues:

147

zhfeng commented 8 months ago

Thanks @mcruzdev for this contribution. Please ping me if it is reay for reviewing.

mcruzdev commented 8 months ago

@zhfeng Surely, I am testing locally here!

mcruzdev commented 8 months ago

Hi @zhfeng, I think that you can review now.

mcruzdev commented 8 months ago

Hi @zhfeng I will make some changes here, I will use the approach one sidecar per service first, to be more simple, because it is the behavior default for dapr.

salaboy commented 8 months ago

This is awesome! I hope we can review and merge this soon!

zhfeng commented 8 months ago

@mcruzdev very nice work! and I just add some comments.

Can we only fix one issue per PR? It could be much clearer for the mainainers to understand.

Thanks a lot!

salaboy commented 8 months ago

@zhfeng thanks for the feedback!

zhfeng commented 8 months ago

I will start a new release soon. Thanks all of you!

zhfeng commented 8 months ago

@all-contributors please add @mcruzdev as code!

allcontributors[bot] commented 8 months ago

@zhfeng

I've put up a pull request to add @mcruzdev! :tada: