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

Make Rain Terraces more terse using standard array functions #149

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

Conversation

Bruce-Feldman
Copy link
Contributor

Used library functions on arrays instead of loops.

@codecov-io
Copy link

codecov-io commented Aug 10, 2018

Codecov Report

Merging #149 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #149   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         121    121           
  Lines        2335   2325   -10     
  Branches      396    397    +1     
=====================================
- Hits         2335   2325   -10
Impacted Files Coverage Δ
...thms/uncategorized/rain-terraces/dpRainTerraces.js 100% <100%> (ø) ⬆️

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 70ec623...680ee71. Read the comment docs.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@Bruce-Feldman Bruce-Feldman force-pushed the Tersify-Rain-Terraces branch from a274395 to 680ee71 Compare August 11, 2018 13:54
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.

None yet

2 participants