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

Add a warning on missing pages in pdfly cat #33

Open
Lucas-C opened this issue Oct 13, 2023 · 5 comments
Open

Add a warning on missing pages in pdfly cat #33

Lucas-C opened this issue Oct 13, 2023 · 5 comments

Comments

@Lucas-C
Copy link
Member

Lucas-C commented Oct 13, 2023

We want to produce a warning (on stderr) when a user attemps to use pdfly cat
to extract a range of pages that is outside the number of pages of the PDF document processed.

We already have a unit test regarding this, so this feature could considered DONE when this unit test will pass:
https://github.com/py-pdf/pdfly/blob/main/tests/test_extract_images.py#L20

@pavankalyan224847
Copy link

hello i would like to work on this issue can you please assign it to me

@Lucas-C
Copy link
Member Author

Lucas-C commented Oct 16, 2023

hello i would like to work on this issue can you please assign it to me

Great! Sure, I just did so 🙂

Do you plan to work on this as part of https://hacktoberfest.com?

Also please tell us if you stop working on this, so that we can unassign this issue.

@pavankalyan224847
Copy link

Thank you for assigning me this issue and yes i want to work on this as a part of https://hacktoberfest.com

@Lucas-C
Copy link
Member Author

Lucas-C commented Dec 15, 2023

Hi @pavankalyan224847

Are you still planning to work on this, or can we unassign you?

@pavankalyan224847
Copy link

please unassign this issue i'll comeback later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants