Bacon / BaconQrCode

QR Code Generator for PHP
BSD 2-Clause "Simplified" License
1.83k stars 208 forks source link

Debian adjustments and project improvements #118

Closed williamdes closed 1 year ago

williamdes commented 1 year ago

Since the last version that added a new dependency to run tests I can not package it in Debian, I made one adjustment to allow me to --exclude-group integration and just have to remove use MatchesSnapshots; on Debian.

I also made changes to make everybody happy, exclude the tests from composer archives and allow me to have them in Debian

codecov-commenter commented 1 year ago

Codecov Report

Merging #118 (bc63923) into master (fc62b92) will not change coverage. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #118   +/-   ##
=========================================
  Coverage     62.88%   62.88%           
  Complexity      934      934           
=========================================
  Files            47       47           
  Lines          2727     2727           
=========================================
  Hits           1715     1715           
  Misses         1012     1012           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

DASPRiD commented 1 year ago

LGTM!

DASPRiD commented 1 year ago

Will you need a new release/tag for Debian?

williamdes commented 1 year ago

Will you need a new release/tag for Debian?

Thank you for the merges!

Please, that would be awesome

DASPRiD commented 1 year ago

There you go, released as 2.0.8.

williamdes commented 1 year ago

There you go, released as 2.0.8.

Just for you to know, the snapshot testing library is too complicated to package in terms of dependencies to also package. That's why I disable the integration test :)