Skip to content

Conversation

@joshlory
Copy link
Contributor

Request from @ryansloan -- they cover up the top two rows, making them not usable in the puzzle.

@joshlory joshlory requested a review from Hamms May 30, 2018 19:18
@joshlory
Copy link
Contributor Author

@codecov-io
Copy link

codecov-io commented May 30, 2018

Codecov Report

Merging #11 into master will increase coverage by 0.95%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   39.41%   40.37%   +0.95%     
==========================================
  Files          22       22              
  Lines        1413     1372      -41     
  Branches      231      223       -8     
==========================================
- Hits          557      554       -3     
+ Misses        714      684      -30     
+ Partials      142      134       -8
Impacted Files Coverage Δ
src/beeItemDrawer.js 70.49% <ø> (+23.06%) ⬆️
src/bee.js 24.63% <ø> (+0.86%) ⬆️

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 7f13126...0baa66c. Read the comment docs.

Copy link
Contributor

@Hamms Hamms left a comment

Choose a reason for hiding this comment

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

oh, awesome! I've never liked those.

@joshlory joshlory force-pushed the dont-show-collected-bee-items branch 3 times, most recently from b956369 to 0baa66c Compare May 30, 2018 22:26
@joshlory joshlory merged commit 8092388 into master May 30, 2018
@joshlory joshlory deleted the dont-show-collected-bee-items branch May 30, 2018 22:29
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.

3 participants