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

Added Discovery to grpc #4

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

koendelaat
Copy link

Overview

What this PR does / why we need it

Adds Discovery to the grpc API.

Special notes for your reviewer

@koendelaat
Copy link
Author

@loafoe Can you review?

@loafoe loafoe force-pushed the master branch 2 times, most recently from 4de4cb4 to e9507e8 Compare July 16, 2024 13:13
@loafoe
Copy link
Member

loafoe commented Jul 16, 2024

LGTM

I rebased upstream and this causes a bunch of conflicts, can you resolve. I think you should still keep the PR in upstream open and updated to get it merged. Otherwise we need to resolve this every time there is a PB change in upstream and we rebase :)

@loafoe loafoe force-pushed the master branch 4 times, most recently from 3911ff1 to 246547c Compare July 29, 2024 21:52
@koendelaat koendelaat force-pushed the feature/grpc-discovery branch from 8d0ac8b to 2be966f Compare August 22, 2024 16:39
Signed-off-by: Koen de Laat <[email protected]>
@koendelaat koendelaat force-pushed the feature/grpc-discovery branch from 2be966f to 6bbd1ff Compare August 22, 2024 16:41
@koendelaat
Copy link
Author

@loafoe, please rebase master with upstream, that should resolve the conflicts I guess

@loafoe loafoe force-pushed the master branch 2 times, most recently from 41d6502 to 2919dfd Compare August 23, 2024 13:21
@koendelaat koendelaat force-pushed the feature/grpc-discovery branch from cf4ec26 to 6bbd1ff Compare August 23, 2024 16:38
@loafoe loafoe merged commit ac08ca7 into philips-forks:master Aug 26, 2024
4 checks passed
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.

2 participants