Skip to content

Conversation

Aschen
Copy link
Contributor

@Aschen Aschen commented Jun 14, 2019

What does this PR do?

Proxify module use the object spread operator.
Kuzzle use the Javascript SDK and we keep es5 compat so we have to keep the SDK Javascript es6compatible too.

@Aschen Aschen self-assigned this Jun 14, 2019
Copy link
Contributor

@scottinet scottinet left a comment

Choose a reason for hiding this comment

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

don't we have babel installed?

Copy link
Contributor

@scottinet scottinet left a comment

Choose a reason for hiding this comment

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

Sorry, my mistake, the dist build cannot be used in a Node.js environment 😑

@scottinet scottinet changed the title HOTFIX Remove usage of spread operator Fix incompatibility with ES6 Javascript Jun 14, 2019
@codecov
Copy link

codecov bot commented Jun 14, 2019

Codecov Report

Merging #403 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #403   +/-   ##
=======================================
  Coverage   96.31%   96.31%           
=======================================
  Files          32       32           
  Lines        1519     1519           
=======================================
  Hits         1463     1463           
  Misses         56       56
Impacted Files Coverage Δ
src/proxify.js 95.23% <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 1a72b02...35cf1e9. Read the comment docs.

@scottinet scottinet merged commit 9695c4e into master Jun 14, 2019
@scottinet scottinet deleted the fix-es6-spread-operator-usage branch June 14, 2019 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants