search

jschlyter / ttls

Twinkly Twinkly Little Star
Other
8 stars 6 forks source link

License #39

Closed joostlek closed 4 months ago

joostlek commented 4 months ago

Hey 👋🏻,

I am currently looking into the licenses at Home Assistant and apparently this dependency uses a non-OSI approved license. Please consider using a OSI approved license to support the Twinkly integration.

jschlyter commented 4 months ago

Hmmm, I believe I have a plain 2-clause BSD license, no?

joostlek commented 4 months ago

Copyright (c) 2019 Jakob Schlyter. All rights reserved.

The all rights reserved voids it I think

jschlyter commented 4 months ago

Sure, dropped now.

jschlyter commented 4 months ago

Are we good? I can make another release if needed, otherwise it will appear in the next one.

joostlek commented 4 months ago

Can you also add the classifier License :: OSI Approved :: BSD License to the package? It will help with detecting the license in the future CI check.

joostlek commented 4 months ago

A release would be nice, its fine if you already planned a release, but if the next time is around December, rather have it now just to have it done :)

jschlyter commented 4 months ago

Just need one review of https://github.com/jschlyter/ttls/pull/38 and I'll make the release :-)

jschlyter commented 4 months ago

Released https://github.com/jschlyter/ttls/releases/tag/v1.8.3