piggly-dev / php-pix

Uma biblioteca para preparar e gerar o código Pix do Banco Central do Brasil.
MIT License
91 stars 28 forks source link

Suporte ao PHP 8.x #3

Closed leoneparise closed 3 years ago

leoneparise commented 3 years ago

Esta PR adiciona suporte ao PHP 8.x bem como atualiza algumas bibliotecas e corrige alguns testes.

caiquearaujo commented 3 years ago

Vou analisar esse final de semana e, se tiver tudo certo, faço o merge!

caiquearaujo commented 3 years ago

@leoneparise só precisamos corrigir uma coisa, não dá para usar a chillerlan/php-qrcode na versão ^4.2, pois ela retira o suporte ao PHP 7.2, que deve ser mantido nesta biblioteca. Ou então mudar a linha "php": "^7.2|^8.0" para "php": "^7.4|^8.0"

leoneparise commented 3 years ago

Ola @caiquearaujo, fiz alguma correções como adotar a versão mínima do PHP para 7.4 e o downgrade do phpunit para 8.5. No entanto, devido a biblioteca chillerlan/php-qrcode não é possível dar suporte ao PHP 7.2 e 8.x ao mesmo tempo. A versão 3.x da biblioteca suporta o PHP 7.2 e a 4.x o PHP 7.4 em diante. Caso queria fazer merge, sinta-se a vontate, estou usando a versão que se encontra na minha branch em produção sem nenhum problema.

Muito obrigado pela biblioteca, fez um ótimo trabalho. :)

caiquearaujo commented 3 years ago

Entendo, como mudar a versão da biblioteca chillerlan/php-qrcode cria incompatibilidades com o PHP 7.2, vamos precisar avaliar se é mesmo o melhor momento para essa feature. Pois, seguindo o SimVer vamos precisar migrar para a versão 2.0 nessa biblioteca.

leoneparise commented 3 years ago

Entendo, como mudar a versão da biblioteca chillerlan/php-qrcode cria incompatibilidades com o PHP 7.2, vamos precisar avaliar se é mesmo o melhor momento para essa feature. Pois, seguindo o SimVer vamos precisar migrar para a versão 2.0 nessa biblioteca.

Exatamente, muitas bibliotecas estão fazendo isso para dar suporte ao PHP 8.

caiquearaujo commented 3 years ago

Vou fechar essa pull request para mudar a estrutura dos branches antes de atualizar para a versão 2.x.x da biblioteca.