Skip to content

Conversation

@fhennig
Copy link
Contributor

@fhennig fhennig commented Jun 16, 2022

closes #86

@fhennig fhennig marked this pull request as ready for review June 16, 2022 12:10
Copy link
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

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

Some ideas

fhennig and others added 8 commits June 20, 2022 09:46
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
…ech/documentation into service-discovery-refactoring
Copy link
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

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

Puh, writing docs is hard. I made some suggestions mostly related to form not content.

fhennig and others added 8 commits June 23, 2022 15:19
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
@fhennig fhennig requested a review from razvan June 23, 2022 14:18
@fhennig
Copy link
Contributor Author

fhennig commented Jun 23, 2022

I've added a first paragraph that already gives an overview, I've moved a lot of text into a "Motivation" section to clarify that. I've added some more details to the usage, in an effort to make it clearer. The second sub-paragraph under usage could probably use an example, but I couldn't come up with a good, compact example.

razvan
razvan previously approved these changes Jun 23, 2022
Copy link
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

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

lgtm

Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
fhennig and others added 2 commits June 23, 2022 16:55
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Co-authored-by: Razvan-Daniel Mihai <84674+razvan@users.noreply.github.com>
Copy link
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

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

approved

@fhennig fhennig merged commit 14f3a3d into main Jun 23, 2022
@fhennig fhennig deleted the service-discovery-refactoring branch June 23, 2022 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants