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

FEAT: DPF TIFF Manager Module #875

Draft
wants to merge 3 commits into
base: integration
Choose a base branch
from
Draft

Conversation

carlwilson
Copy link
Member

This is currently a draft PR used to test the build of JHOVE with an OPF DPF based TIFF module.

- TIFF validation based on DPF manager;
 + routine to run validation from DPF classes;
 + map DPF error messages to JHOVE; and
- added the necessary to the JHOVE installer for a new module.
- Added OPF artifactory settings to JHOVE pom.xml.
- Resolved conflicts in installer dependencies.
@carlwilson carlwilson marked this pull request as draft July 27, 2023 11:07
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Patch coverage has no change and project coverage change: +16.86% 🎉

Comparison is base (8a4d1ce) 30.15% compared to head (e096515) 47.02%.
Report is 3 commits behind head on integration.

Additional details and impacted files
@@                Coverage Diff                 @@
##             integration     #875       +/-   ##
==================================================
+ Coverage          30.15%   47.02%   +16.86%     
- Complexity           651     1101      +450     
==================================================
  Files                 57       57               
  Lines               9079     9079               
  Branches            1622     1622               
==================================================
+ Hits                2738     4269     +1531     
+ Misses              5965     4279     -1686     
- Partials             376      531      +155     

see 34 files with indirect coverage changes

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

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.

1 participant