Skip to content

Add man pages for zstdgrep and zstdless#1372

Merged
Cyan4973 merged 2 commits into
facebook:devfrom
samrussell:dev
Oct 16, 2018
Merged

Add man pages for zstdgrep and zstdless#1372
Cyan4973 merged 2 commits into
facebook:devfrom
samrussell:dev

Conversation

@samrussell

Copy link
Copy Markdown
Contributor

zgrep and zless come with man pages, this adds the same to zstdgrep and zstdless

The zstdgrep man page closely mirrors the zgrep man page; the zstdless is a little... less

Comment thread programs/zstdless.1.md Outdated
SYNOPSIS
--------

`zless` [*flags*] [_file_ ...]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should it be zstdless ?

Comment thread programs/zstdgrep.1.md Outdated
SYNOPSIS
--------

`zgrep` [*grep-flags*] [--] _pattern_ [_files_ ...]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should it be zstdgrep ?

@Cyan4973

Copy link
Copy Markdown
Contributor

Thanks @samrussell , it's a welcomed doc addition !

Just a couple of comments, as I suspect some left-over from zgrep/zless documentation.

@samrussell

Copy link
Copy Markdown
Contributor Author

Thanks @Cyan4973, got a bit wordblind at the end, should be no more zgrep/zless references

@Cyan4973

Copy link
Copy Markdown
Contributor

Thanks for the patch @samrussell !
Will be part of next release !

@Cyan4973 Cyan4973 merged commit 374f2b4 into facebook:dev Oct 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants