Skip to content
This repository was archived by the owner on Mar 10, 2020. It is now read-only.

chore: use Object.assign instead of spread operator #913

Merged
merged 2 commits into from
Dec 14, 2018

Conversation

fsdiogo
Copy link
Contributor

@fsdiogo fsdiogo commented Dec 14, 2018

This way the requiring lib doesn't have to support the spread operator.

@fsdiogo fsdiogo requested a review from alanshaw December 14, 2018 12:06
@ghost ghost assigned fsdiogo Dec 14, 2018
@ghost ghost added the in progress label Dec 14, 2018
Copy link
Contributor

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

For future, commit message should have been prefixed "refactor:" not "chore:".

@ghost ghost assigned alanshaw Dec 14, 2018
@alanshaw
Copy link
Contributor

@vasco-santos would you mind looking into the failure here https://ci.ipfs.team/blue/organizations/jenkins/IPFS%2Fjs-ipfs-http-client/detail/PR-913/3/pipeline/58

I'm going to merge this as the test failures are completely unrelated.

@alanshaw alanshaw merged commit 330d9e7 into master Dec 14, 2018
@alanshaw alanshaw deleted the chore/use-object-assign-instead-of-spread branch December 14, 2018 16:05
@ghost ghost removed the in progress label Dec 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants