search

gmcquistin / react-throttle

80 stars 8 forks source link

Persist throttle/debounce arguments #1

Closed gmcquistin closed 8 years ago

gmcquistin commented 8 years ago

Resolves #2.

I have copied throttle and debounce from the Lodash library and added a line which clones the arguments, so the debounced function will receive a clone of the original arguments. In this way, even if the original caller modifies the arguments after the fact, we will still have a copy.

codecov-io commented 8 years ago

Current coverage is 87.27%

Merging #1 into master will decrease coverage by -12.73% as of c58767c

@@            master      #1   diff @@
======================================
  Files            5       7     +2
  Stmts           30     110    +80
  Branches         5      23    +18
  Methods          0       0       
======================================
+ Hit             30      96    +66
  Partial          0       0       
- Missed           0      14    +14

Review entire Coverage Diff as of c58767c

Powered by Codecov. Updated on successful CI builds.

liddellj commented 8 years ago

LGTM :+1: