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

Simplify InlineParameterFormatterTest #1411

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Mar 6, 2024

Q A
Type improvement
BC Break no
Fixed issues N/A

Summary

This PR follows #1408 and simplifies our test setup by bumping DBAL a bit.

@derrabus derrabus merged commit c8f23d8 into doctrine:3.8.x Mar 6, 2024
12 checks passed
@derrabus derrabus deleted the improvement/inline-parameter-formatter-test branch March 6, 2024 13:54
derrabus added a commit to derrabus/migrations that referenced this pull request Mar 11, 2024
* 3.8.x:
  Raise PHPStan level to 8 (doctrine#1409)
  Remove unused paramaters from DiffCommand and VersionCommand (doctrine#1386)
  Simplify InlineParameterFormatterTest (doctrine#1411)
  fix: Allow enum param types: ArrayParameterType and ParameterType (doctrine#1408)
derrabus added a commit to derrabus/migrations that referenced this pull request Mar 11, 2024
* 3.8.x:
  Raise PHPStan level to 8 (doctrine#1409)
  Remove unused paramaters from DiffCommand and VersionCommand (doctrine#1386)
  Simplify InlineParameterFormatterTest (doctrine#1411)
  fix: Allow enum param types: ArrayParameterType and ParameterType (doctrine#1408)
derrabus added a commit to derrabus/migrations that referenced this pull request Mar 11, 2024
* 3.8.x:
  Fix CS
  Raise PHPStan level to 8 (doctrine#1409)
  Remove unused paramaters from DiffCommand and VersionCommand (doctrine#1386)
  Simplify InlineParameterFormatterTest (doctrine#1411)
  fix: Allow enum param types: ArrayParameterType and ParameterType (doctrine#1408)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants