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

cpdf: add page #3713

Merged
merged 4 commits into from
Dec 31, 2019
Merged

cpdf: add page #3713

merged 4 commits into from
Dec 31, 2019

Conversation

andrik
Copy link
Collaborator

@andrik andrik commented Dec 30, 2019

  • The page (if new), does not already exist in the repo.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

Hi guys, this one is from #3689 I'll try to create other ones. As the cheat project allows more examples then we do, I decided to pick up what looks like the most common ones.

@tldr-bot
Copy link

The build
for this PR failed with the following error(s):

Error: Parse error on line 17:
...to/document.pdf}}``- Split a document 
----------------------^
Expecting 'BACKTICK', 'COMMAND_TEXT', 'COMMAND_TOKEN', got 'NEWLINE'
pages/common/cpdf.md:6: TLDR002 A single space should precede a sentence

Please fix the error(s) and push again.

@tldr-bot
Copy link

The build
for this PR failed with the following error(s):

pages/common/cpdf.md:6: TLDR002 A single space should precede a sentence
pages/common/cpdf.md:22: TLDR002 A single space should precede a sentence
pages/common/cpdf.md:26: TLDR002 A single space should precede a sentence

Please fix the error(s) and push again.

@andrik andrik added the new command Issues requesting creation of a new page. label Dec 30, 2019
Copy link
Member

@einverne einverne left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution @andrik . Good to know this useful command. I left a small suggestion for you to take a look.

pages/common/cpdf.md Outdated Show resolved Hide resolved
Copy link
Contributor

@zlatanvasovic zlatanvasovic left a comment

Choose a reason for hiding this comment

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

LGTM after @einverne's concerns are applied.

@ivanhercaz ivanhercaz merged commit db65c5b into tldr-pages:master Dec 31, 2019
Copy link
Member

@agnivade agnivade left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. Please take a look at the comments.


- Split a document into ten-page chunks, writing them to chunk001.pdf, chunk002.pdf, etc:

`cpdf -split {{path/to/document.pdf}} -o {{path/to/chunk%%%.pdf}} -chunk 10`
Copy link
Member

Choose a reason for hiding this comment

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

10 should be in tokens ({{}})


- Encrypt a document using 128bit encryption, providing `fred` as owner password and `joe` as user password:

`cpdf -encrypt 128bit fred joe {{path/to/source_document.pdf}} -o {{path/to/encrypted_document.pdf}}`
Copy link
Member

Choose a reason for hiding this comment

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

128bit, fred and joe should be in tokens ({{}})


- Decrypt a document using the owner password `fred`:

`cpdf -decrypt {{path/to/encrypted_document.pdf}} owner=fred -o {{path/to/decrypted_document.pdf}}`
Copy link
Member

Choose a reason for hiding this comment

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

fred should be in tokens ({{}})


`cpdf -list-annotations {{path/to/document.pdf}}`

- Create a new document, with metadata, based on an existing one:
Copy link
Member

Choose a reason for hiding this comment

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

Let's change this to "Create a new document from an existing one with additional metadata"

> CLI to manipulate existing PDF files in a variety of ways.
> Homepage: <https://www.coherentpdf.com/cpdfmanual/cpdfmanual.html>.

- Select pages 1, 2, 3 and 6 from source document and write those to a destination document:
Copy link
Member

Choose a reason for hiding this comment

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

from a source document.


- Select pages 1, 2, 3 and 6 from source document and write those to a destination document:

`cpdf {{path/to/source_document.pdf}} 1-3,6 -o {{path/to/destination_document.pdf}}`
Copy link
Member

Choose a reason for hiding this comment

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

1-3,6 should be in tokens ({{}})

Copy link
Collaborator

@ivanhercaz ivanhercaz left a comment

Choose a reason for hiding this comment

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

I translated this page to Spanish (#3717), so I reviewed it by myself.

# cpdf

> CLI to manipulate existing PDF files in a variety of ways.
> Homepage: <https://www.coherentpdf.com/cpdfmanual/cpdfmanual.html>.
Copy link
Collaborator

Choose a reason for hiding this comment

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

After review it again to translate it to Spanish, I think the most consistent way to put an URL for the command is "More information". Although I haven't read a guideline or a rule about it, all the pages I remember to read have "More information". Should we unify the line that provides the URL?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, that's right. We should use "More information"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new command Issues requesting creation of a new page.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants