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.

Sign up for GitHub

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

Jump to bottom

Fix Laravel-Horizon and PHP-Worker Build Issue #2952

Merged
merged 2 commits into from Apr 17, 2021
Merged

Fix Laravel-Horizon and PHP-Worker Build Issue #2952

merged 2 commits into from Apr 17, 2021

Conversation

bestlong
Copy link
Member

fix #2828

@bestlong bestlong merged commit b3cdfd5 into master Apr 17, 2021
@ManuDoni
Copy link

Now it is better than before but I don't think this fixes #2828, it just hides the error behind a .env variable

@bestlong bestlong deleted the fix-2828 branch April 20, 2021 10:18
@bestlong
Copy link
Member Author

@ManuDoni

i was review CI build result https://github.com/laradock/laradock/runs/2372033058?check_suite_focus=true

seem all fine. need more info.

@ManuDoni
Copy link

@bestlong yes you're right.
I also see here the specific line in which Redis gets downloaded.
Sorry for disturbing you, maybe it's an error affecting only me but I saw others having problems using pecl with php-alpine based docker image. I don't know how to solve it, I think I'll open an issue.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Laravel-Horizon and PHP-Worker Build Issue
2 participants
@bestlong @ManuDoni