python-pdfkit icon indicating copy to clipboard operation
python-pdfkit copied to clipboard

Raise exceptions control

Open avpeshkov opened this issue 3 years ago • 5 comments
trafficstars

Hi all, I've been working with pdfkit and found several cases where wkhtmltopdf outputs errors, but still generates a pdf. I would like to be able to manually control how the pdf-kit responds to errors.

ps: I apologize if I violate protocol in any way, this is my first time doing pull request to opensource.

avpeshkov avatar Nov 29 '21 12:11 avpeshkov

Hi, thanks for the work! Not sure about it though, since when using this option any errors will be ignored, even when pdf is not generated. More correct way is to check if we got any output and if there are errors - we can ignore those, and if output is missing - properly raise an exception

JazzCore avatar Nov 30 '21 10:11 JazzCore

Hi, thanks for the work! Not sure about it though, since when using this option any errors will be ignored, even when pdf is not generated. More correct way is to check if we got any output and if there are errors - we can ignore those, and if output is missing - properly raise an exception

Hey, Thanks for the quick review! I have taken your remarks into account and updated pull request, also I added info about the changes to README.

avpeshkov avatar Nov 30 '21 15:11 avpeshkov

Hi, again:) I fixed tests to get python2 compatibility. Who would have thought there would be such a mistake in workflow:)

avpeshkov avatar Dec 02 '21 17:12 avpeshkov

Looks like GH CI is broken atm... No time to review at the moment, sorry. I'll try to test it on this weekend

JazzCore avatar Dec 03 '21 09:12 JazzCore

Thank you, this errors looks vary unusual. No problem, we all have a job:)

avpeshkov avatar Dec 03 '21 12:12 avpeshkov