-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Update pihole-FTL man-pages #3883
Conversation
Thanks for that PR, but note all PRs should be filed against |
I also adjusted the description of the arguments to be consistent with the other sub-commands and updated the man-page date that I forgot before. |
Is there still anything missing? |
You'll need to actually rebase on |
Ensure that all configuration options currently recognized by FTL 5.3.1 are properly documented in the man page of the configuration file. Signed-off-by: Reto Gantenbein <reto.gantenbein@linuxmonk.ch>
Signed-off-by: Reto Gantenbein <reto.gantenbein@linuxmonk.ch>
Signed-off-by: Reto Gantenbein <reto.gantenbein@linuxmonk.ch>
Signed-off-by: Reto Gantenbein <reto.gantenbein@linuxmonk.ch>
b100757
to
ebad56e
Compare
Thanks, I didn't get that. Done now. |
Thanks, can you also squash the 4 commits? It helps keep things clean. |
I'll squash it for him. |
This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there: https://discourse.pi-hole.net/t/pi-hole-core-web-v5-2-2-and-ftl-v5-3-3-released/41998/1 |
By submitting this pull request, I confirm the following:
git rebase
)What does this PR aim to accomplish?:
Ensure that all (configuration) options currently recognized by FTL 5.3.1 are properly documented in their corresponding man-pages.
How does this PR accomplish the above?:
Adding the missing configuration options read from currently released FTL's src/config.c as well as the runtime options read from the output of
pihole-FTL --help
What documentation changes (if any) are needed to support this PR?:
This is a documentation change.