-
-
Notifications
You must be signed in to change notification settings - Fork 581
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
Fix update command for v6 #3299
Conversation
Signed-off-by: XhmikosR <[email protected]>
This depends on the user running the command really. Does the Debian documentation call out that you have to run Oh, and since the template wasn't filled out and the PR description doesn't say anything useful, this PR simply prepends the word |
I just saw pi-hole/pi-hole#6066 and thought I'd submit a PR with the change, but feel free to close. I know that for me at least, it needs |
Maybe we could create a documentation page similar to this one explaining how to add users to
|
Hmmm, thinking more about this - would a user being in the Ultimately I think this is a bit of a non-issue. We transparently elevated to Either way, it's fairly self documented. As I mentioned in my comment on pi-hole/pi-hole#6070 (comment) - Other services may not even be as helpful as suggesting to rerun the command with |
I agree. |
Then I'd say maybe just drop |
Thank you for your contribution to the Pi-hole Community!
Please read the comments below to help us consider your Pull Request.
We are all volunteers and completing the process outlined will help us review your commits quicker.
Please make sure you
What does this PR aim to accomplish?:
How does this PR accomplish the above?:
Link documentation PRs if any are needed to support this PR:
By submitting this pull request, I confirm the following:
git rebase
)