Added arrow function in below files #4061

saravyas · 2018-11-14T04:18:49Z

This PR includes arrow function change.

attempt.js

jdalton · 2018-11-14T05:06:25Z

I'm actually fine with these being function declarations though the removal of .apply() is fine. I know it kind of changes the intent of the PR though.

conformsTo.js

conforms.js

cond.js

compact.js

cloneWith.js

before.js

cloneDeepWith.js

cloneDeep.js

clone.js

clamp.js

chunk.js

castArray.js

before.js

attempt.js

jdalton · 2018-11-18T03:34:03Z

Thank you for your patience and follow-ups @saravyas!

lock · 2019-11-18T03:38:04Z

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

saravyas added 7 commits November 13, 2018 12:17

converted to fat arrow function

e2658db

added export func in at.js

422171f

added arrow func in attempt.js

8e0d36f

kept single quote in at.js

97153ce

added arrow func in castarray.js

9c841d8

added arrow func in chunk.js

2450754

added arrow func in 10 files

4bf0551

jdalton added the enhancement label Nov 14, 2018

jdalton reviewed Nov 14, 2018

View reviewed changes

attempt.js Show resolved Hide resolved

added spread operator

bb7a759

jdalton reviewed Nov 14, 2018

View reviewed changes

conformsTo.js Outdated Show resolved Hide resolved

jdalton reviewed Nov 14, 2018

View reviewed changes

conforms.js Outdated Show resolved Hide resolved

jdalton reviewed Nov 14, 2018

View reviewed changes

cond.js Outdated Show resolved Hide resolved

jdalton reviewed Nov 14, 2018

View reviewed changes

compact.js Outdated Show resolved Hide resolved

jdalton reviewed Nov 14, 2018

View reviewed changes

cloneWith.js Outdated Show resolved Hide resolved

jdalton reviewed Nov 14, 2018

View reviewed changes

before.js Outdated Show resolved Hide resolved

saravyas added 2 commits November 14, 2018 11:47

kept normal func for this

0274d9a

change request

b377a34