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

Remove warnings for PDF/A, PDF/UA compatibility. #2014

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

m000
Copy link
Contributor

@m000 m000 commented Dec 7, 2023

Avoids spamming production logs with PDF/A, PDF/UA compatibility warnings.

@liZe
Copy link
Member

liZe commented Dec 8, 2023

Hi!

Thanks for this pull request.

Actually, I think that you can even remove these messages. We’ve probably spotted most of the outstanding bug for now, some issues have been opened but it looks like it works well enough that people can actually generate PDF/A and /UA for real-life cases.

Could you please change the PR to actually remove these messages?

Support for the standards works well enough at this point.
@m000 m000 force-pushed the pdfa-pdfua-warn-once-pr branch from 92f070d to 1bae13c Compare December 11, 2023 18:10
@m000
Copy link
Contributor Author

m000 commented Dec 11, 2023

Hi!

Thanks for this pull request.

Actually, I think that you can even remove these messages. We’ve probably spotted most of the outstanding bug for now, some issues have been opened but it looks like it works well enough that people can actually generate PDF/A and /UA for real-life cases.

Could you please change the PR to actually remove these messages?

Easy-peasy. Done & thanks!

@m000 m000 changed the title Only warn once for PDF/A, PDF/UA compatibility. Remove warnings for PDF/A, PDF/UA compatibility. Dec 11, 2023
@liZe liZe added this to the 61.0 milestone Dec 11, 2023
@liZe
Copy link
Member

liZe commented Dec 11, 2023

Thank you!

@liZe liZe merged commit e8f1418 into Kozea:main Dec 11, 2023
6 checks passed
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