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.

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

Copyedit README.md from --chomp to end of file #64

Merged

Conversation

rolandwalker
Copy link
Contributor

I tried to keep the revisions minimal, as before.

One sentence I did not understand was:

Specifying from one line to another is a typical use case for this option.

It is left unchanged, but probably should be made more clear.

README.md Outdated Show resolved Hide resolved
@rolandwalker rolandwalker force-pushed the RW/copyedit-readme-from-chomp-to-end branch from 4aca6a3 to 47c8926 Compare January 1, 2024 20:07
@greymd
Copy link
Owner

greymd commented Jan 1, 2024

Thank you so much. Your contribution is quite helpful since I am not a native :)

Specifying from one line to another is a typical use case for this option.

It is left unchanged, but probably should be made more clear.

The purpose of this statement is to emphasize that the -z option is useful for treating multiple lines as a single input. For instance, it enables the targeted command to process the contents from line A to line B inclusively as one unified input.

I have reviewed the sentence and agree that the current explanation is somewhat difficult to understand. I'm considering revising it as follows:

Before:
Specifying from one line to another is a typical use case for this option.
After:
This option is useful for treating multiple lines as a single combined input.

Ho do you think about it?

@rolandwalker
Copy link
Contributor Author

This option is useful for treating multiple lines as a single combined input.

Very clear!

@greymd greymd merged commit c5e42bd into greymd:main Jan 1, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants