Merge pull request #11223 from dtrodrigues/update-guidelines

docs: remove recommended/optional from maintainer guidelines
This commit is contained in:
Dustin Rodrigues 2021-04-23 08:03:04 -04:00 committed by GitHub
commit d4a2abec7e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -31,8 +31,7 @@ This is all that really matters:
- Thank people for contributing. - Thank people for contributing.
Checking dependencies is important, because they will probably stick around Checking dependencies is important, because they will probably stick around
forever. Nobody really checks if they are necessary or not. Use the forever. Nobody really checks if they are necessary or not.
`:optional` and `:recommended` modifiers as appropriate.
Depend on as little stuff as possible. Disable X11 functionality if possible. Depend on as little stuff as possible. Disable X11 functionality if possible.
For example, we build Wireshark, but not the heavy GUI. For example, we build Wireshark, but not the heavy GUI.