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

Remove halo.pl! #36

Merged
merged 8 commits into from Oct 24, 2014
Merged

Remove halo.pl! #36

merged 8 commits into from Oct 24, 2014

Conversation

@Lordnibbler
Copy link
Owner

Lordnibbler commented Aug 18, 2014

  • TRY sending a unique ID with each message, log each message in beaglebone client and see if they are being received more than once
  • Remove halo.pl
  • Remove unnecessary logging for speed (or set production logging level lower)
  • send json/rgb object instead of weird formatted 000,000,000,000\n string
  • fix tests
@Lordnibbler Lordnibbler force-pushed the halo branch 4 times, most recently from c52ceb6 to 5706f58 Aug 23, 2014
Lordnibbler added a commit that referenced this pull request Oct 24, 2014
Remove halo.pl!
@Lordnibbler Lordnibbler merged commit 097afe4 into master Oct 24, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@Lordnibbler Lordnibbler deleted the halo branch Oct 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.