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

Multiple files / folders feature added #6

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

jseg380
Copy link
Contributor

@jseg380 jseg380 commented Sep 15, 2023

Added functionality for passing multiple files at once.

Now files and folders can be passed as arguments, PDF files will be cleaned if they exist, otherwise a message error will be printed and the program execution will continue. If a folder is passed as an argument, the program will look for PDF files and folders, it will clean the files and repeat this process for the folders, thus, searching for PDF files recursively.

For the sake of avoiding unnecessarily long print messages when cleaning multiple files at once, the "short" option (-s) has been added not to show metadata about cleaned files.

@YM162 YM162 self-assigned this Sep 19, 2023
@YM162
Copy link
Owner

YM162 commented Sep 19, 2023

Hi!
Thank you for your contribution, I'll review it ASAP :)

Copy link
Owner

@YM162 YM162 left a comment

Choose a reason for hiding this comment

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

LGTM

@YM162 YM162 merged commit 9090208 into YM162:main Sep 19, 2023
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.

None yet

2 participants