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

MAINT: remove unused code #2805

Merged
merged 3 commits into from
Aug 22, 2024
Merged

MAINT: remove unused code #2805

merged 3 commits into from
Aug 22, 2024

Conversation

pubpub-zz
Copy link
Collaborator

@pubpub-zz pubpub-zz commented Aug 20, 2024

remove _get_page from reader which is redundant with get_page
faced again some issues with a broken link from tika. Replace them with github links.

@pubpub-zz pubpub-zz marked this pull request as draft August 20, 2024 13:10
@pubpub-zz
Copy link
Collaborator Author

pubpub-zz commented Aug 20, 2024

tika files (links are dissapearing progressively):

@stefan6419846
Copy link
Collaborator

tika files (links are dissapearing progressively)

Is there any official indication about this?

Copy link

codecov bot commented Aug 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.85%. Comparing base (0c81f3c) to head (36226e1).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2805   +/-   ##
=======================================
  Coverage   95.85%   95.85%           
=======================================
  Files          51       51           
  Lines        8579     8573    -6     
  Branches     1696     1695    -1     
=======================================
- Hits         8223     8218    -5     
  Misses        212      212           
+ Partials      144      143    -1     

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

@pubpub-zz
Copy link
Collaborator Author

pubpub-zz commented Aug 20, 2024

@pubpub-zz
Copy link
Collaborator Author

tika files (links are dissapearing progressively)

Is there any official indication about this?

No. I've just observer à New linked reported as dead. World you prefer to do it on a dead link report basis?

@stefan6419846
Copy link
Collaborator

I tend to prefer this to be done only if there is real need to do so. Additionally, it might make sense as well to review the files before just adding them. There must be reasons why they have been deleted and I am not always sure whether we really need them or whether we could replace them with another one.

@pubpub-zz
Copy link
Collaborator Author

Additionally, it might make sense as well to review the files before just adding them.

All thèse files have been added based on different PRs added for years. @MartinThoma reported them.
Today they provide a good pool of test file

There must be reasons why they have been deleted and I am not always sure whether we really need them or whether we could replace them with another one.

@MartinThoma do you have any idea why links become dead?
Providing replacement is a hudge job. These where reported from a major apache 2.0 project. The licence analysis was for me good enough.

This reverts commit 90cca33.
@pubpub-zz pubpub-zz marked this pull request as ready for review August 20, 2024 21:04
@pubpub-zz pubpub-zz changed the title MAINT: remove unused code + replace links to tika MAINT: remove unused code Aug 21, 2024
@stefan6419846 stefan6419846 merged commit d2d520b into py-pdf:main Aug 22, 2024
17 checks passed
@pubpub-zz pubpub-zz mentioned this pull request Sep 15, 2024
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.

2 participants