-
Notifications
You must be signed in to change notification settings - Fork 628
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JavaScript: handle spread syntax #3846
JavaScript: handle spread syntax #3846
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3846 +/- ##
=======================================
Coverage 85.02% 85.02%
=======================================
Files 228 228
Lines 53897 53900 +3
=======================================
+ Hits 45828 45831 +3
Misses 8069 8069 ☔ View full report in Codecov by Sentry. |
I must add a reference to a page describing the spread operator. |
@masatake our GitHub Actions BSD workflows alwasy fail these days. I have no idea how to fix it. The only way is waiting GitHub Actions Team's fix. Reference: actions/runner-images#8730 |
@leleliu008 Thank you. Could you make a "temporarily disabling" pull request and merge it? |
Done via #3848 |
It seems that "spread operator" is a term in TypeScript. |
With this change, the parser skip "..." and the subsequent expression. ref. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Spread_syntax Related to universal-ctags#3435. Signed-off-by: Masatake YAMATO <[email protected]>
f07356d
to
014fbcd
Compare
Related to #3435.