mergeWithDotPath point-free version #35

aizerin · 2018-01-30T07:35:34Z

closes #22

tommmyy

Thanks to you we have 3 more useful functions 👍

tommmyy · 2018-01-30T08:28:12Z

src/mergeWithDotPath.js

-	path,
-	mergeFn(dotPath(path, obj), value), obj)
-);
+const takeFirst = nthArg(0);


call it firstArg and make it standalone function

tommmyy · 2018-01-30T08:29:07Z

src/mergeWithDotPath.js

-	mergeFn(dotPath(path, obj), value), obj)
-);
+const takeFirst = nthArg(0);
+const takeForth = nthArg(3);


call it lastArg

implement it like following: nthArg(-1)

and make it standalone function

aizerin · 2018-01-30T16:43:24Z

Done. I've added headArg and lastArg. I didn't write tests for them because I think that is not necessary.

tommmyy reviewed Jan 30, 2018

View reviewed changes

aizerin force-pushed the point_free_merge_dot_path branch from ede65ed to 7e2364e Compare January 30, 2018 16:36

aizerin added 2 commits January 30, 2018 17:40

added headArg and lastArg functions

5de6731

mergeWithDotPath point-free version

fd792c2

aizerin force-pushed the point_free_merge_dot_path branch from 7e2364e to fd792c2 Compare January 30, 2018 16:40

tommmyy merged commit d0fe5b6 into tommmyy:master Jan 30, 2018

Provide feedback