-
Notifications
You must be signed in to change notification settings - Fork 39
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
doc: describe the default behaviour #313
Conversation
Because for someone new, it might not be clear what it does. The idea is also to modify the welcome page on mptcpd.mptcp.dev with the same description. Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Pull Request Test Coverage Report for Build 11889725129Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks @matttbe!
While describing what's being done by default, best to mention how to adapt that. Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Thank you for the review @ossama-othman! I was going to merge this PR, but then I realised that while at it, it might be good to describe how to change the behaviour. So I just added a new paragraph about that if that's OK with you :) |
@matttbe Sure! No problem at all. Thanks! |
Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Signed-off-by: Matthieu Baerts (NGI0) <[email protected]>
Because for someone new, it might not be clear what it does.
The idea is also to modify the welcome page on mptcpd.mptcp.dev with the same description.
Note that I also modified the main wiki page with the same description.