-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 typos in tests/ #9242
Fix typos in tests/ #9242
Conversation
tests/zfs-tests/tests/functional/userquota/userquota_005_neg.ksh
Outdated
Show resolved
Hide resolved
tests/zfs-tests/tests/functional/userquota/userquota_010_pos.ksh
Outdated
Show resolved
Hide resolved
tests/zfs-tests/tests/functional/userquota/userquota_005_neg.ksh
Outdated
Show resolved
Hide resolved
tests/zfs-tests/tests/functional/userquota/userquota_010_pos.ksh
Outdated
Show resolved
Hide resolved
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.
This looks good, though I submitted a couple optional comments.
tests/zfs-tests/tests/functional/userquota/userquota_005_neg.ksh
Outdated
Show resolved
Hide resolved
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.
Changes requested in other comments.
tests/zfs-tests/tests/functional/userquota/userquota_005_neg.ksh
Outdated
Show resolved
Hide resolved
Thanks a lot for patch review to @freqlabs and @rlaager Signed-off-by: Andrea Gelmini <[email protected]>
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.
Thank you very much!
Codecov Report
@@ Coverage Diff @@
## master #9242 +/- ##
==========================================
- Coverage 79.31% 77.49% -1.83%
==========================================
Files 400 389 -11
Lines 122034 121539 -495
==========================================
- Hits 96792 94185 -2607
- Misses 25242 27354 +2112
Continue to review full report at Codecov.
|
Reviewed-by: Ryan Moeller <[email protected]> Reviewed-by: Richard Laager <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Andrea Gelmini <[email protected]> Closes openzfs#9242
Reviewed-by: Ryan Moeller <[email protected]> Reviewed-by: Richard Laager <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Andrea Gelmini <[email protected]> Closes openzfs#9242
Reviewed-by: Ryan Moeller <[email protected]> Reviewed-by: Richard Laager <[email protected]> Reviewed-by: Brian Behlendorf <[email protected]> Signed-off-by: Andrea Gelmini <[email protected]> Closes #9242
No description provided.