py-pdf / pdfly

CLI tool to extract (meta)data from PDF and manipulate PDF files
BSD 3-Clause "New" or "Revised" License
108 stars 18 forks source link

Add warning for out-of-bounds page range in pdfly cat command #58

Closed Zingzy closed 1 month ago

Zingzy commented 1 month ago

PR Description

This PR addresses issue #33 where users attempting to use the pdfly cat command to extract a range of pages outside the number of pages in the PDF document do not receive a warning. The following changes have been made:

Testing

Lucas-C commented 1 month ago

Hi @Zingzy

This looks great, thank your for your contribution 👍

Could you just run black pdfly/cat.py and include the changes in this PR? Currently the GitHub Actions pipeline fails because this code formatter has not been applied to this file following your changes: https://github.com/py-pdf/pdfly/actions/runs/11204446524/job/31171421427?pr=58

Zingzy commented 1 month ago

Hello @Lucas-C, I made the necessary changes. 😄

Zingzy commented 1 month ago

Hello @Lucas-C any updates on this?

Lucas-C commented 1 month ago

I merged your MR!

Thank you very much for your contribution @Zingzy 👍