zephir-lang / php-zephir-parser

The Zephir Parser delivered as a C extension for the PHP language.
https://zephir-lang.com
MIT License
185 stars 36 forks source link

#118 - Add support for `yield` statement #122

Closed Jeckerson closed 3 years ago

Jeckerson commented 3 years ago

Hello!

In raising this pull request, I confirm the following:

Thanks

codecov[bot] commented 3 years ago

Codecov Report

Merging #122 (1dc8955) into development (5605563) will increase coverage by 0.47%. The diff coverage is 100.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #122      +/-   ##
===============================================
+ Coverage        75.32%   75.79%   +0.47%     
===============================================
  Files                5        5              
  Lines             2926     2954      +28     
===============================================
+ Hits              2204     2239      +35     
+ Misses             722      715       -7     
Flag Coverage Δ
unittests 75.79% <100.00%> (+0.47%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
parser/base.c 93.86% <100.00%> (+0.03%) :arrow_up:
parser/parser.h 89.52% <100.00%> (+0.45%) :arrow_up:
parser/scanner.re 98.28% <100.00%> (+<0.01%) :arrow_up:
parser/zephir.lemon 48.44% <100.00%> (+0.88%) :arrow_up:

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 5605563...1dc8955. Read the comment docs.

AlexNDRmac commented 3 years ago

@Jeckerson can you add a test with lambda func, please.