-
Notifications
You must be signed in to change notification settings - Fork 325
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
[docs] update package, features and config mode docs #3188
Conversation
@skorpy2009 would you like to have a look at this, please? |
Pleases prefix and separate the commits by topic. |
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.
not a complete review, just what i noticed at a glance
2d67bc2
to
940b810
Compare
I would like to get this merged :) |
* document network roles and their effects
Add information about old x86 images
I update the PR to reflect to latest changes of the web-cellular package in #3307 The build of the resulting docs can be found here: |
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.
i'd merge it, if you revert your wlan/wifi change.
wifi is a trademark and we want to use wlan.
if there is something like "wifi", please change that one in turn to "wlan", but not the other way around.
Co-authored-by: Tom Herbers <github@herbetom.de>
Thanks @rotanid - I removed this commit through a rebase. |
This is a series of docs improvements created from #3131
I tested the docs using
make -C docs html && python -m http.server --directory docs/_build/html
.I am open for additions and suggestions.
If I should split the commits into separate PRs I can sure do that too, though I did not want to create 5 PRs about docs.
I think that especially some visualizations to refer to are quiet important for users too, so I added two pictures.