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

1493 laravel update to 10 #1530

Merged
merged 5 commits into from
Mar 24, 2023
Merged

Conversation

vasilysmolin
Copy link
Contributor

@vasilysmolin vasilysmolin commented Mar 23, 2023

laravel update from 9 to 10
closes #1493

@ssssank ssssank requested a review from fey March 24, 2023 05:15
@fey
Copy link
Collaborator

fey commented Mar 24, 2023

Для запуска Raco у нас используется пакет от Symfony.
Бахни туда туду, чтобы перейти на новый компонент лары.

@vasilysmolin
Copy link
Contributor Author

Для запуска Raco у нас используется пакет от Symfony. Бахни туда туду, чтобы перейти на новый компонент лары.

можешь подсказать куда именно todo поставить? Я просто с raco не знаком)

@fey
Copy link
Collaborator

fey commented Mar 24, 2023

@vasilysmolin надо глянуть где у нас использует пакет симфони для запуска внешних процессов.

@vasilysmolin
Copy link
Contributor Author

@fey вот здесь только вижу

$command = new Command("raco test --check-stderr --quiet {$solutionPath}");

@fey
Copy link
Collaborator

fey commented Mar 24, 2023

@vasilysmolin ага

@vasilysmolin
Copy link
Contributor Author

@fey готово

@fey fey merged commit b3b1580 into Hexlet:main Mar 24, 2023
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.

Upgrade to Laravel 10
2 participants