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

Apache 2 licensing and attribution #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adamkempenich
Copy link

Hi Joshua! I was scrolling Github looking for Hue driver, and while scrolling, I noticed that you're using some of my MagicHome code in your project verbatim to my original writing in your project.

This is fine—and it's completely fair game to use—but it's licensed under Apache 2 and requires attribution. Specifically, the imports, logDebug preferences and method, deviceIP preferences, parts of your commands ( '// 0 - 255' ), and parse, are verbatim to my original code.

It looks like this is the only file affected, so I've made a PR with attribution added in hopes that you'll work with me in honoring the Apache 2 license listed in my drivers.

Hi Joshua! I was scrolling Github looking for Hue driver, and while scrolling, I noticed that you're using some of my MagicHome code in your project verbatim to my original writing in your project. 

This is fine—and it's completely fair game to use—but it's licensed under Apache 2 and requires attribution. Specifically, the imports, logDebug preferences and method,  deviceIP preferences, parts of your commands ( '// 0 - 255' ), and parse, are verbatim to my original code.

It looks like this is the only file affected, so I've made a PR with attribution added in hopes that you'll work with me in honoring the Apache 2 license listed in my drivers.
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.

1 participant