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

phpstan - return to production ver #976

Merged
merged 1 commit into from
Dec 11, 2024
Merged

Conversation

drbyte
Copy link
Contributor

@drbyte drbyte commented Dec 10, 2024

The ...-x-dev version was only used temporarily during an older PR 960 to get tests to pass.

This PR simply resets it to normal.

Described at: #960 (comment)

Note: phpstan is only used by the tests, and is not required for Mobile-Detect to work.

The `...-x-dev` version was only used temporarily during an older PR 960 to get tests to pass.

This PR simply resets it to normal.

Described at: serbanghita#960 (comment)
@drbyte
Copy link
Contributor Author

drbyte commented Dec 10, 2024

It appears that Github Actions failed to properly download a functional phpunit package at the time this PR was submitted.

Manually rerunning the tests will probably allow them to pass.

@serbanghita
Copy link
Owner

thanks for this, I'll manually check it on local and retry the CI build

@serbanghita
Copy link
Owner

@drbyte it looks like I also forgot to create a Docker specific run for vendor/bin/phpstan analyse src tests
In CI we run this, but not on Docker (for local dev). Can you add[0] and test this, if not I'll add it later?

[0] https://github.com/serbanghita/Mobile-Detect/blob/4.8.x/docker-compose.yml

@serbanghita serbanghita self-assigned this Dec 11, 2024
@serbanghita serbanghita merged commit 5989711 into serbanghita:4.8.x Dec 11, 2024
5 checks passed
@drbyte drbyte deleted the patch-1 branch December 12, 2024 04:14
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.

2 participants