Skip to content

Conversation

scottinet
Copy link
Contributor

Description

Babel.js is not able to handle the async/await feature without a polyfill to be included along with the SDK on production.
Forcing the use of a polyfill is not a desired outcome for our SDK, so async/await is now a banned feature.

How-to test

The "raw web" getting started tests are currently failing on the documentation because of this problem. Using this branch version solve the failing test.

@scottinet scottinet self-assigned this Feb 6, 2019
@codecov-io
Copy link

codecov-io commented Feb 6, 2019

Codecov Report

Merging #368 into 6-dev will increase coverage by <.01%.
The diff coverage is 85.71%.

Impacted file tree graph

@@            Coverage Diff            @@
##           6-dev     #368      +/-   ##
=========================================
+ Coverage   96.6%   96.61%   +<.01%     
=========================================
  Files         28       28              
  Lines       1415     1417       +2     
=========================================
+ Hits        1367     1369       +2     
  Misses        48       48
Impacted Files Coverage Δ
src/Kuzzle.js 95.41% <85.71%> (+0.04%) ⬆️

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 70904a5...63fa5ed. Read the comment docs.

@alexandrebouthinon alexandrebouthinon merged commit e21b115 into 6-dev Feb 6, 2019
@alexandrebouthinon alexandrebouthinon deleted the remove-async-await branch February 6, 2019 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants