Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Rewrite update without lodash #120

Merged
merged 3 commits into from
Oct 30, 2017
Merged

Rewrite update without lodash #120

merged 3 commits into from
Oct 30, 2017

Conversation

frinyvonnick
Copy link
Contributor

Prerequisites

Description

Rewrite update without lodash

Issue : #109

@frinyvonnick frinyvonnick requested a review from nlepage as a code owner October 30, 2017 12:56
@codecov-io
Copy link

codecov-io commented Oct 30, 2017

Codecov Report

Merging #120 into feature/107 will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           feature/107   #120   +/-   ##
==========================================
  Coverage          100%   100%           
==========================================
  Files               73     74    +1     
  Lines              243    245    +2     
  Branches            47     47           
==========================================
+ Hits               243    245    +2
Impacted Files Coverage Δ
src/util/convert.js 100% <100%> (ø) ⬆️
src/object/update.js 100% <100%> (ø) ⬆️
src/core/update.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 b03007c...81e5d11. Read the comment docs.

@nlepage nlepage force-pushed the without-lodash/update branch 3 times, most recently from 37eb01f to 4a15896 Compare October 30, 2017 14:55

const inc = (v, i = 1) => v + i

const add = (v, ...additions) => additions.reduce((prev, next) => prev + next, v)
Copy link
Member

Choose a reason for hiding this comment

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

The word addends would be more accurate than additions

@nlepage nlepage force-pushed the without-lodash/update branch from 2612e48 to 81e5d11 Compare October 30, 2017 18:56
@nlepage nlepage self-assigned this Oct 30, 2017
@nlepage nlepage changed the base branch from feature/107 to master October 30, 2017 18:57
@nlepage nlepage merged commit 921786c into master Oct 30, 2017
@nlepage nlepage deleted the without-lodash/update branch October 30, 2017 18:57
@nlepage nlepage added this to the 0.4-alpha milestone Oct 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants