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

General Fixes and Improvements to lighting #55

Merged
merged 1 commit into from
Feb 15, 2017
Merged

Conversation

T1mothyyang
Copy link
Member

More lighting effects
Abstracted more parts of TwoWayComm

@T1mothyyang T1mothyyang force-pushed the ligthing-fixes branch 2 times, most recently from c5b164d to a99220f Compare February 15, 2017 02:22
More lighting effects
Abstracted more parts of TwoWayComm
@codecov
Copy link

codecov bot commented Feb 15, 2017

Codecov Report

Merging #55 into master will decrease coverage by -0.07%.
The diff coverage is 62.5%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   62.79%   62.72%   -0.07%     
==========================================
  Files          51       51              
  Lines         758      762       +4     
  Branches       59       62       +3     
==========================================
+ Hits          476      478       +2     
- Misses        282      284       +2
Impacted Files Coverage Δ
...krobotics/potassium/lighting/RXTXTwoWayComms.scala 23.25% <ø> (ø)
...obotics/potassium/lighting/LightingComponent.scala 75% <71.42%> (-25%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d112447...1888a3c. Read the comment docs.

Copy link
Contributor

@shadaj shadaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shadaj shadaj merged commit 02d682e into master Feb 15, 2017
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