Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Atualização para Laravel 10 #77

Closed
emtudo opened this issue Feb 14, 2023 · 4 comments · Fixed by #81 or #82
Closed

Atualização para Laravel 10 #77

emtudo opened this issue Feb 14, 2023 · 4 comments · Fixed by #81 or #82
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@emtudo
Copy link
Contributor

emtudo commented Feb 14, 2023

Pelos meus testes funciona na versão php 8.2 sem alteração alguma.

E alterei o composer.json e o phpunit.xml e funcionou também para laravel 10.

Você pretende fazer a alteração? Posso fazer PR?

o phpunit.xml praticamente foi criado com o comando: vendor/bin/phpunit --migrate-configuration para funcioanr

O composer.json só adicionei as versões nova do laravel e pronto.

@emtudo emtudo added documentation Improvements or additions to documentation enhancement New feature or request labels Feb 14, 2023
@lsnepomuceno
Copy link
Owner

lsnepomuceno commented Feb 14, 2023

Estou validando alguns pontos, pois irei depreciar o uso do symfony/process para dar uso do ProcessFacade do Laravel 10, como você pode ver no link:
https://laravel-news.com/process-facade-laravel-10

Como o Laravel 10 foi lançado hoje, imagino que até o final de semana já finalize os testes e suba uma nova versão.

@emtudo
Copy link
Contributor Author

emtudo commented Feb 14, 2023

Eu já usei o progress do symfony, o laravel está apenas usando o mesmo fazendo um facade como vc mencionou, isso vc poderia lançar e depois fazer atualização pq não vai mudar nada para quem usa o pacote no final das contas.

@emtudo
Copy link
Contributor Author

emtudo commented Feb 14, 2023

Na verdade eu até preferia que vc não usasses o facade do laravel rs

Pq ficaria mais simples portar o pacote para usar no symfony. Como seria bom se ele fosse mais agnóstico.

This was linked to pull requests Feb 16, 2023
@lsnepomuceno
Copy link
Owner

Suporte ao Laravel 10 liberado a partir da versão 1.0.6.

A depreciação do symfony/process ocorrerá a partir da futura v2 deste pacote que terá suporte ao Laravel 11.

Repository owner locked as resolved and limited conversation to collaborators Feb 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
2 participants