-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Be consistent with use of the "sudo" - v2 #9568
Conversation
Included the "sudo" command to commands that need to be executed with the super-user privilege.
- Added `Important` notes to inform users the `sudo` command is required to be installed in order to properly follow instructions. Ticket [5720](https://redmine.openinfosecfoundation.org/issues/5720)
NOTE: This PR may contain new authors:
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #9568 +/- ##
==========================================
- Coverage 82.20% 82.20% -0.01%
==========================================
Files 968 968
Lines 274275 274275
==========================================
- Hits 225461 225460 -1
- Misses 48814 48815 +1
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
Hi!
Thank you for your first contribution to our project. :)
- I believe the changes requested in the previous PR here: Install: Be consistent with use of the "sudo" - v1 #9552 (comment) are yet to be incorporated in this one.
- Please keep your commit history clean.
@@ -90,18 +90,20 @@ Rust support:: | |||
Ubuntu/Debian | |||
""""""""""""" | |||
|
|||
.. Important:: The following instructions require `sudo` be installed. |
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.
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 found it on a link(rst docs on sourceforge) I was directed to via Sphinx's docs.
And I saw that .. Important::
defines that the instruction has to be followed for the work to be successful.
But I'll change it to .. note::
now.
Thanks.
Make sure these boxes are signed before submitting your Pull Request -- thank you.
Link to redmine ticket: https://redmine.openinfosecfoundation.org/issues/5720
Previous PR: #9552
Describe changes:
Important
notes to inform users it is required to installsudo
in order to successfully follow instructions.sudo
commands added to some instructions wheresudo
was not needed.