Skip to content

Prevent loop when babel can transpile but goja can't parse#1651

Merged
mstoykov merged 3 commits intomasterfrom
preventInfiniteLoopOnCompile
Oct 6, 2020
Merged

Prevent loop when babel can transpile but goja can't parse#1651
mstoykov merged 3 commits intomasterfrom
preventInfiniteLoopOnCompile

Conversation

@mstoykov
Copy link
Contributor

@mstoykov mstoykov commented Oct 6, 2020

No description provided.

@mstoykov mstoykov added this to the v0.29.0 milestone Oct 6, 2020
@mstoykov mstoykov requested review from imiric and na-- October 6, 2020 09:06
na--
na-- previously approved these changes Oct 6, 2020
Copy link
Contributor

@na-- na-- left a comment

Choose a reason for hiding this comment

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

LGTM 🤦‍♂️ 😅

@codecov-commenter
Copy link

codecov-commenter commented Oct 6, 2020

Codecov Report

Merging #1651 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1651      +/-   ##
==========================================
- Coverage   72.16%   72.15%   -0.01%     
==========================================
  Files         166      166              
  Lines       12787    12787              
==========================================
- Hits         9228     9227       -1     
+ Misses       3009     3008       -1     
- Partials      550      552       +2     
Impacted Files Coverage Δ
js/compiler/compiler.go 76.47% <100.00%> (ø)
lib/execution.go 89.32% <0.00%> (-2.92%) ⬇️
lib/executor/vu_handle.go 95.23% <0.00%> (+1.90%) ⬆️

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 9b75efd...8ba64d9. Read the comment docs.

na--
na-- previously approved these changes Oct 6, 2020
@mstoykov mstoykov merged commit ef717e1 into master Oct 6, 2020
@mstoykov mstoykov deleted the preventInfiniteLoopOnCompile branch October 6, 2020 12:54
mstoykov added a commit that referenced this pull request Nov 17, 2020
This is continuation of PR #1651 as it turns out that sometimes babel
doesn't do enough on the first time it runs over some scripts. So let it
run twice.
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