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 1.10.26 #6116

Merged
merged 1 commit into from
Jul 27, 2023
Merged

PHPStan 1.10.26 #6116

merged 1 commit into from
Jul 27, 2023

Conversation

derrabus
Copy link
Member

Q A
Type improvement
Fixed issues N/A

Summary

Apparently, the stubs for db2_num_rows() were changed upstream. I'm ignoring the error for now, maybe we can fix it on 3.7.x if we feel like it's worth doing.

@derrabus derrabus added this to the 3.6.6 milestone Jul 27, 2023
@derrabus derrabus merged commit 195aad3 into doctrine:3.6.x Jul 27, 2023
@derrabus derrabus deleted the bump/phpstan branch July 27, 2023 17:57
derrabus added a commit to derrabus/dbal that referenced this pull request Aug 8, 2023
* 3.6.x:
  Run tests with MySQL 8.1 (doctrine#6118)
  Fix a typo and a space (doctrine#6119)
  PHPStan 1.10.26 (doctrine#6116)
  Fix self deprecation
derrabus added a commit to derrabus/dbal that referenced this pull request Aug 8, 2023
* 3.7.x:
  Run tests with MySQL 8.1 (doctrine#6118)
  Sync MariaDB support strategy with upstream (doctrine#6110)
  Fix a typo and a space (doctrine#6119)
  PHPStan 1.10.26 (doctrine#6116)
  Fix self deprecation
  Pin Slevomat Coding Standard
cgknx pushed a commit to cgknx/dbal that referenced this pull request Aug 30, 2023
|      Q       |   A
|------------- | -----------
| Type         | improvement
| Fixed issues | N/A

#### Summary

Apparently, the stubs for `db2_num_rows()` were changed upstream. I'm
ignoring the error for now, maybe we can fix it on 3.7.x if we feel like
it's worth doing.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants