labelle-org / labelle

Label printing software
Apache License 2.0
27 stars 4 forks source link

Add deprecation errors for old dymoprint arguments #35

Closed maresb closed 3 months ago

maresb commented 3 months ago

You're right it's probably not worth the effort, but I was thinking of tweaking the deprecation message in dymoprint to mention that Labelle v1 is a drop-in replacement for dymoprint.

tomers commented 3 months ago

Let's go for it then, but I would like to remove these legacy entries when we release a new major version with breaking changes.

maresb commented 3 months ago

Let's go for it then, but I would like to remove these legacy entries when we release a new major version with breaking changes.

Absolutely. v2 should be a clean break. We can simply revert this commit. This was also somewhat of a sanity check to ensure that we're not missing anything, and indeed it uncovered #33 and #34.