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 rm funcionality #59

Merged
merged 5 commits into from
Oct 22, 2024
Merged

Add rm funcionality #59

merged 5 commits into from
Oct 22, 2024

Conversation

ebotiab
Copy link
Contributor

@ebotiab ebotiab commented Oct 15, 2024

Address #53

@ebotiab
Copy link
Contributor Author

ebotiab commented Oct 15, 2024

Hello @Lucas-C could you review my PR?

tests/test_rm.py Outdated Show resolved Hide resolved
pdfly/cat.py Outdated Show resolved Hide resolved
pdfly/cli.py Outdated Show resolved Hide resolved
pdfly/cat.py Outdated Show resolved Hide resolved
pdfly/cli.py Outdated Show resolved Hide resolved
@Lucas-C Lucas-C merged commit ad0c33d into py-pdf:main Oct 22, 2024
6 checks passed
@Lucas-C
Copy link
Member

Lucas-C commented Oct 22, 2024

Thank you for your contribution @ebotiab 👍

I merged this MR

MartinThoma added a commit that referenced this pull request Dec 8, 2024
## What's new

### New Features (ENH)
- New `booklet` command to adjust offsets and lengths ([PR #77](#77))
- New `uncompress` command ([PR #75](#75))
- New `update-offsets` command to adjust offsets and lengths ([PR #15](#15))
- New `rm` command ([PR #59](#59))
- `metadata`: now also displaying CreationDate, Creator, Keywords & Subject ([PR #73](#73))
- Add warning for out-of-bounds page range in pdfly `cat` command ([PR #58](#58))

### Bug Fixes (BUG)
- `2-up` command, that only showed one page per sheet, on the left side, with blank space on the right ([PR #78](#78))

[Full Changelog](0.3.3...0.4.0)
@Lucas-C
Copy link
Member

Lucas-C commented Dec 8, 2024

This has been released in version 0.4.0: https://pypi.org/project/pdfly/#history

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.

2 participants