Fix typo in zfs-module-parameters man page #6054

Merged
merged 1 commit into from Apr 24, 2017

Conversation

Contributor

bunder2015 commented Apr 21, 2017

Description

Fix typo in zfs-module-parameters man page

Motivation and Context

Makes my eyes not bleed

How Has This Been Tested?

None should be necessary as it's a man page.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the ZFS on Linux code style requirements.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • All commit messages are properly formatted and contain Signed-off-by.
  • Change has been approved by a ZFS on Linux member.
Fix typo in zfs-module-parameters man page
Fix typo in zfs-module-parameters man page

Signed-off-by: bunder2015 <omfgbunder@gmail.com>

ryao approved these changes Apr 22, 2017

ptx0 approved these changes Apr 22, 2017

ptx0 commented Apr 22, 2017

this needs more reviewers.

Member

ryao commented Apr 23, 2017

@ptx0 Good point.

@ahrens @tuxoko @nedbass @dweeezil @prakashsurya @ironMann @rlaager @dun @dajhorn @gunnarbeutner @tonyhutter @tcaputi @don-brady Would any of you care to contribute to a potential attempt at breaking the most reviewed patch in OpenZFS record? ;)

Looks right 👍 😄

LGTM 👍

LGTM! And, yes, it took quite a stare to see what changed!

@behlendorf behlendorf merged commit 603a178 into zfsonlinux:master Apr 24, 2017

19 of 22 checks passed

buildbot/Ubuntu 14.04 i686 (TEST) Build done.
Details
buildbot/Ubuntu 16.04 x86_64 (STYLE) Build done.
Details
buildbot/Ubuntu 16.04 x86_64 Kmemleak (TEST) Build done.
Details
buildbot/Amazon 2015.09 x86_64 (BUILD) Build done.
Details
buildbot/Amazon 2015.09 x86_64 Release (TEST) Build done.
Details
buildbot/CentOS 6.7 x86_64 (BUILD) Build done.
Details
buildbot/CentOS 6.7 x86_64 (TEST) Build done.
Details
buildbot/CentOS 7 x86_64 Mainline (TEST) Build done.
Details
buildbot/CentOS 7.1 x86_64 (BUILD) Build done.
Details
buildbot/CentOS 7.1 x86_64 (TEST) Build done.
Details
buildbot/Debian 8 arm (BUILD) Build done.
Details
buildbot/Debian 8 ppc (BUILD) Build done.
Details
buildbot/Debian 8 ppc64 (BUILD) Build done.
Details
buildbot/Debian 8 x86_64 (BUILD) Build done.
Details
buildbot/Fedora 24 x86_64 (BUILD) Build done.
Details
buildbot/Kernel.org Built-in x86_64 (BUILD) Build done.
Details
buildbot/Kernel.org Default x86_64 (BUILD) Build done.
Details
buildbot/Ubuntu 14.04 i686 (BUILD) Build done.
Details
buildbot/Ubuntu 14.04 x86_64 (BUILD) Build done.
Details
buildbot/Ubuntu 14.04 x86_64 (TEST) Build done.
Details
buildbot/Ubuntu 16.04 aarch64 (BUILD) Build done.
Details
buildbot/Ubuntu 16.04 x86_64 (BUILD) Build done.
Details
Owner

behlendorf commented Apr 24, 2017

Merged, we'll have to make due with only 27 reviews! Thanks.

Member

ryao commented Apr 24, 2017

I can say that this patch had the most of any that I have ever seen. It probably broke the old record. Someone will need to check to see if any patches have more reviewers to confirm. I'll be happy to let someone else volunteer to do that though.

@bunder2015 bunder2015 deleted the bunder2015:patch-1 branch Apr 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment