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

MySensors.pm - MySensors controller module for Misterhouse #613

Merged
merged 1 commit into from
Aug 14, 2016

Conversation

jsiddall
Copy link
Contributor

@jsiddall jsiddall commented Aug 14, 2016

I am not a programmer so creating this was a bit of a stretch for me. As this is a single file it should be harmless for anyone who doesn't "use MySensors". It works fine on my machine and it's tidied with POD included so should be fairly readable. I know there were a few others on the mailing list interested in MySensors so this might be a benefit to some. There are a couple of other failed pull attempts with some bypassing git steps that ended badly. Those can be deleted and/or ignored.

Let me know if you have any suggestions.

Thanks!

@hollie
Copy link
Owner

hollie commented Aug 14, 2016

Hey @jsiddall you did perfectly fine.

Thanks for contributing the code. I'll merge your pull request right away.

Kind regards,
Lieven

@hollie hollie merged commit e28cccd into hollie:master Aug 14, 2016
@jsiddall
Copy link
Contributor Author

On 2016-08-14 02:12, Lieven Hollevoet wrote:

Hey @jsiddall [1] you did perfectly fine.

Thanks for contributing the code. I'll merge your pull request right
away.

Great, thanks for the quick reply!

Jeff

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.

2 participants