Skip to content

Conversation

lionello
Copy link
Member

@lionello lionello commented Apr 4, 2025

@lionello lionello requested review from raphaeltm and commit111 April 4, 2025 18:05
Copy link
Contributor

@commit111 commit111 left a comment

Choose a reason for hiding this comment

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

looks good to me, just a minor comment.

Copy link
Member

@jordanstephens jordanstephens left a comment

Choose a reason for hiding this comment

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

Quick thoughts.

@lionello lionello changed the title Use default network More networking Apr 11, 2025
@lionello lionello requested a review from jordanstephens April 11, 2025 18:56
@lionello lionello changed the title More networking More networking docs Apr 11, 2025
Copy link
Member

@jordanstephens jordanstephens left a comment

Choose a reason for hiding this comment

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

A couple more small suggestion inline, but I think this is a great improvement 👍

@lionello lionello merged commit 6aa58d7 into main Apr 11, 2025
2 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.

4 participants