Skip to content

ENH: Add support for IndirectObject.__contains__#3155

Merged
stefan6419846 merged 2 commits intopy-pdf:mainfrom
noamkush:indirect-object-contains
Mar 4, 2025
Merged

ENH: Add support for IndirectObject.__contains__#3155
stefan6419846 merged 2 commits intopy-pdf:mainfrom
noamkush:indirect-object-contains

Conversation

@noamkush
Copy link
Copy Markdown
Contributor

Fixed #3154

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.53%. Comparing base (231f7a3) to head (ae4d02a).
Report is 104 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3155      +/-   ##
==========================================
+ Coverage   96.51%   96.53%   +0.01%     
==========================================
  Files          53       53              
  Lines        8875     8918      +43     
  Branches     1622     1635      +13     
==========================================
+ Hits         8566     8609      +43     
  Misses        185      185              
  Partials      124      124              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stefan6419846
Copy link
Copy Markdown
Collaborator

Thanks for the PR. I do not really think that this is a bug, but rather a new feature. Additionally, could you please provide a positive test as well?

@noamkush
Copy link
Copy Markdown
Contributor Author

noamkush commented Mar 4, 2025

Sure, I added a positive test. BTW, some javascript tests already test for in.

@stefan6419846 stefan6419846 changed the title BUG: IndirectObject __contains__ raises an error. ENH: Add support for IndirectObject.__contains__ Mar 4, 2025
Copy link
Copy Markdown
Collaborator

@stefan6419846 stefan6419846 left a comment

Choose a reason for hiding this comment

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

Thanks.

@stefan6419846 stefan6419846 merged commit 8d16885 into py-pdf:main Mar 4, 2025
17 checks passed
stefan6419846 added a commit that referenced this pull request Mar 16, 2025
## What's new

### New Features (ENH)
- Add support for `IndirectObject.__contains__` (#3155) by @noamkush

### Bug Fixes (BUG)
- Fix detection of inline images followed by names or numbers (#3173) by @stefan6419846

### Robustness (ROB)
- Consider root objects without catalog type as fallback (#3175) by @stefan6419846
- Raise proper error on infinite loop when reading objects (#3169) by @stefan6419846

### Documentation (DOC)
- Mention memory consumption of text extraction (#3168) by @stefan6419846

### Developer Experience (DEV)
- Upgrade to ruff 0.10.0 (#3191) by @stefan6419846

[Full Changelog](5.3.1...5.4.0)
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.

IndirectObject gives a KeyError when using in operator

2 participants