TCPDF icon indicating copy to clipboard operation
TCPDF copied to clipboard

Normalize indentation and apply basic PSR2 CS ruleset

Open mvorisek opened this issue 3 years ago • 5 comments

there should be no code change, thus fully compatible with PHP 5

mvorisek avatar Apr 03 '21 11:04 mvorisek

Codecov Report

Merging #358 (af37dbe) into main (c979d00) will decrease coverage by 0.01%. The diff coverage is 76.58%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #358      +/-   ##
==========================================
- Coverage   72.80%   72.78%   -0.02%     
==========================================
  Files         125      125              
  Lines       22964    22951      -13     
==========================================
- Hits        16718    16705      -13     
  Misses       6246     6246              
Flag Coverage Δ
php-5.3-ubuntu-latest 72.26% <76.42%> (-0.02%) :arrow_down:
php-5.4-ubuntu-latest ?
php-5.5-ubuntu-latest ?
php-5.6-ubuntu-latest 45.11% <68.59%> (-0.03%) :arrow_down:
php-7.1-ubuntu-latest 45.69% <68.32%> (+<0.01%) :arrow_up:
php-7.2-ubuntu-latest 45.64% <68.32%> (ø)
php-7.3-ubuntu-latest 45.63% <68.26%> (+<0.01%) :arrow_up:
php-7.4-ubuntu-latest 45.63% <68.26%> (ø)
php-8.0-ubuntu-latest 65.19% <73.63%> (ø)

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

Impacted Files Coverage Δ
fonts/cid0jp.php 100.00% <ø> (ø)
fonts/dejavusans.php 100.00% <ø> (ø)
fonts/dejavusansb.php 100.00% <ø> (ø)
fonts/dejavusansi.php 100.00% <ø> (ø)
fonts/dejavuserif.php 100.00% <ø> (ø)
fonts/freeserif.php 100.00% <ø> (ø)
fonts/uni2cid_aj16.php 100.00% <ø> (ø)
include/barcodes/pdf417.php 60.58% <ø> (-0.15%) :arrow_down:
include/barcodes/qrcode.php 64.18% <ø> (-0.04%) :arrow_down:
include/tcpdf_font_data.php 100.00% <ø> (ø)
... and 123 more

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 c979d00...af37dbe. Read the comment docs.

codecov-io avatar Apr 03 '21 11:04 codecov-io

Can you please resolve the conflicts?

nicolaasuni avatar Jul 10 '21 07:07 nicolaasuni

Codecov Report

Merging #358 (634a56e) into main (150e756) will decrease coverage by 27.26%. The diff coverage is 68.32%.

:exclamation: Current head 634a56e differs from pull request most recent head 523848e. Consider uploading reports for the commit 523848e to get more accurate results Impacted file tree graph

@@             Coverage Diff             @@
##             main     #358       +/-   ##
===========================================
- Coverage   72.93%   45.67%   -27.27%     
===========================================
  Files         127       90       -37     
  Lines       23024    20291     -2733     
===========================================
- Hits        16792     9267     -7525     
- Misses       6232    11024     +4792     
Flag Coverage Δ
php-5.3-ubuntu-latest ?
php-5.4-ubuntu-latest ?
php-5.5-ubuntu-latest ?
php-5.6-ubuntu-latest ?
php-7.1-ubuntu-latest 45.67% <68.32%> (+<0.01%) :arrow_up:
php-7.2-ubuntu-latest 45.62% <68.32%> (ø)
php-7.3-ubuntu-latest 45.61% <68.26%> (ø)
php-7.4-ubuntu-latest ?
php-8.0-ubuntu-latest ?

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

Impacted Files Coverage Δ
fonts/cid0jp.php 100.00% <ø> (ø)
fonts/dejavusans.php 100.00% <ø> (ø)
fonts/freeserif.php 100.00% <ø> (ø)
fonts/uni2cid_aj16.php 100.00% <ø> (ø)
include/barcodes/pdf417.php 62.12% <ø> (+1.39%) :arrow_up:
include/barcodes/qrcode.php 63.88% <ø> (-0.34%) :arrow_down:
include/tcpdf_font_data.php 100.00% <ø> (ø)
include/tcpdf_fonts.php 35.84% <ø> (-8.93%) :arrow_down:
include/tcpdf_static.php 35.46% <ø> (-18.08%) :arrow_down:
tcpdf.php 39.35% <ø> (-31.53%) :arrow_down:
... and 102 more

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 150e756...523848e. Read the comment docs.

codecov-commenter avatar Jul 10 '21 11:07 codecov-commenter

Can you please resolve the conflicts?

@nicolaasuni done, please merge asap to prevent further conflicts, thanks!

mvorisek avatar Jul 10 '21 11:07 mvorisek

Can you rebase @mvorisek ?

williamdes avatar Sep 06 '23 14:09 williamdes