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

Widen symfony/process requirements #53

Merged
merged 1 commit into from
Jun 18, 2023
Merged

Widen symfony/process requirements #53

merged 1 commit into from
Jun 18, 2023

Conversation

func0der
Copy link
Contributor

No description provided.

@icanhazstring
Copy link
Owner

Seems like all tests are failing now 🤔

@func0der
Copy link
Contributor Author

Would they work beforehand? Can you start one for the current main branch?
I figure that having this --ignore-platform-req=php would ignore php version during composer install no matter which version is running.
And the error messages seems to come out of phpunit.

@icanhazstring
Copy link
Owner

Could you quickly rebase with latest main? Maybe it works. Otherwise I will check later 🙂

@icanhazstring
Copy link
Owner

Ok seems to have fixed some. Will check later 🙂

@icanhazstring icanhazstring merged commit 5c4d379 into icanhazstring:master Jun 18, 2023
6 of 7 checks passed
@func0der
Copy link
Contributor Author

It seems to just fail randomly 🤷
Thanks for merging :)

Are you going to release a new version too?

@func0der func0der deleted the patch-1 branch June 18, 2023 05:23
@icanhazstring
Copy link
Owner

Yep will do 🙂

@func0der
Copy link
Contributor Author

when?

@icanhazstring
Copy link
Owner

Was a bit busy last week. Will see to it 🙂

@icanhazstring
Copy link
Owner

Tagged as 0.8.3

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.

None yet

2 participants