Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix typos in zpool_do_scrub() error messages #17028

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Fix typos in zpool_do_scrub() error messages #17028

merged 1 commit into from
Feb 5, 2025

Conversation

0mp
Copy link
Contributor

@0mp 0mp commented Feb 5, 2025

Sponsored-by: Wasabi Technology, Inc.
Sponsored-by: Klara, Inc.

Motivation and Context

This changes fixes some typos in the error messages in zpool_do_scrub().

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)
  • Library ABI change (libzfs, libzfs_core, libnvpair, libuutil and libzfsbootenv)
  • Documentation (a change to man pages or other documentation)

Checklist:

@0mp
Fix typos in zpool_do_scrub() error messages
71920db 
Sponsored-by: Wasabi Technology, Inc.
Sponsored-by: Klara, Inc.

Signed-off-by: Mateusz Piotrowski <[email protected]>
@amotin amotin added the Status: Accepted Ready to integrate (reviewed, tested) label Feb 5, 2025
@gamanakis
Copy link
Contributor

Mistaken :), Looks good to me

@0mp
Copy link
Contributor Author

0mp commented Feb 5, 2025

Mistaken :), Looks good to me

:D Thanks!

@tonyhutter tonyhutter merged commit 2cccbac into openzfs:master Feb 5, 2025
24 of 25 checks passed
@0mp
Copy link
Contributor Author

0mp commented Feb 6, 2025

Thanks!

@0mp 0mp deleted the patch-1 branch February 6, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gmelikov gmelikov gmelikov approved these changes

@amotin amotin amotin approved these changes

@gamanakis gamanakis gamanakis approved these changes

@tonyhutter tonyhutter tonyhutter approved these changes

Assignees
No one assigned
Labels
Status: Accepted Ready to integrate (reviewed, tested)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@0mp @gamanakis @gmelikov @amotin @tonyhutter