Skip to content

Conversation

@MaximilianKresse
Copy link
Contributor

@MaximilianKresse MaximilianKresse commented Jun 3, 2020

See #48

@MaximilianKresse MaximilianKresse changed the title Added Fpdf output #48 Added Fpdf output Jun 3, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jun 3, 2020

Codecov Report

Merging #49 into master will decrease coverage by 3.87%.
The diff coverage is 0.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##              master      #49      +/-   ##
=============================================
- Coverage     100.00%   96.12%   -3.88%     
- Complexity       312      325      +13     
=============================================
  Files             16       17       +1     
  Lines            720      749      +29     
=============================================
  Hits             720      720              
- Misses             0       29      +29     
Impacted Files Coverage Δ Complexity Δ
src/Output/QRFpdf.php 0.00% <0.00%> (ø) 13.00 <13.00> (?)
src/QRCode.php 100.00% <ø> (ø) 29.00 <0.00> (ø)

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 7e1c765...c7a7016. Read the comment docs.

@codemasher
Copy link
Member

Hey! Thank you so much for your contribution! It looks good so far, only a few minor things (code style, declare, file docblock) i'd fix after merge if that's ok for you.

@MaximilianKresse
Copy link
Contributor Author

Sure :-)

@codemasher codemasher merged commit 5e6c029 into chillerlan:master Jun 3, 2020
codemasher added a commit that referenced this pull request Jun 3, 2020
codemasher added a commit that referenced this pull request Jun 4, 2020
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.

3 participants