discovery-tecnologia/dsc-mercado-livre

Suporte para PHP 8.x?

Opened this issue · 3 comments

Suporte para PHP 8.x?

Opa, e ae @rafa-acioly blz?
Então, gostaria hehe 😅
Mas eu sinceramente estou sem tempo e também um pouco desatualizado das novas versões do PHP.

Mas talvez possamos abrir uma discussão nesse tópico e ver se alguém gostaria de contribuir.

Outra coisa que pensei no passado era mudar um pouco a forma que é construída essa lib.
Hoje ela é um "pouco mais chata" de dar manutenção devido o mapeamento dos Objetos. Pensei usar o mesmo conceito que usei nessa lib skyhub-client - onde uso mais Arrays do que Objetos (mas não estou totalmente certo ainda)

Feedbacks serão bem vindos...

@dilowagner eu posso contribuir com essa mudança, você acha que é melhor disponibilizar como uma nova release ou como a versão padrão? Na minha opinião deveria ser a versão principal.

A lib de teste phpunit esta com uma dependencia forte nas libs do doctrine, o que faz com que as atualizações sejam mais custosas, o que acha de migrarmos para a lib pest?

Com relação a retornar arrays ao invés de objetos, eu pessoalmente prefiro objetos por esses motivos:

  • Confiança em acessar os valores sem correr o risco de ver um erro de index out of range
  • A IDE consegue auto-completar e analisar melhor os atributos (tipos, nomes, auto-navegação, etc)
  • Podemos também colocar atributos como nullable com o operador ?

Criar entidades como classes realmente da mais trabalho mas vale a pena por conta da experiencia de usar a lib.

Perfeito @rafa-acioly
Obrigado por querer contribuir!

Acredito que devemos mudar a versão principal mesmo, iniciar uma V3, certo?
Sobre a troca do framework de teste, não vejo problemas... Inclusive não conhecia esse pest 😅

Arrays x Objetos
Boa, sinceramente não tinha pensado nesses pontos... apenas na manutenção de código
Principalmente, porque podemos nos beneficiar com os novos recursos do PHP 8 (algumas já existiam no 7 hehe).

Totalmente de acordo 👍🏻