computerlyrik / dymoprint

Linux Software to print with LabelManager PnP from Dymo
Apache License 2.0
151 stars 51 forks source link

Some more fixes and improvements #103

Closed tomers closed 8 months ago

maresb commented 8 months ago

Do you know already how to fix the 255 problem?

maresb commented 8 months ago

For pre-commit please see https://github.com/tomers/dymoprint/pull/1

maresb commented 8 months ago

You don't need to exclude devcontainer.json because I switched the pre-commit hook. Please see https://github.com/tomers/dymoprint/pull/1 before you do any work to fix any of the errors, I fix them all there (except for the 255 error).

tomers commented 8 months ago

Do you know already how to fix the 255 problem?

Fixed. Thanks!

maresb commented 8 months ago

Amazing!!! BTW I really like your commit style with lots of nice and short commits. Makes it a breeze to review!