Skip to content

Bump PHPStan and Psalm#214

Merged
greg0ire merged 3 commits intodoctrine:2.2.xfrom
drupol:patch-1
Dec 2, 2021
Merged

Bump PHPStan and Psalm#214
greg0ire merged 3 commits intodoctrine:2.2.xfrom
drupol:patch-1

Conversation

@drupol
Copy link
Contributor

@drupol drupol commented Dec 2, 2021

No description provided.

@stof
Copy link
Member

stof commented Dec 2, 2021

IIRC, using fixed versions rather than a range was meant to avoid getting new violations reported randomly in PRs not related to upgrading phpstan or psalm due to the CI picking a new version silently.

@drupol
Copy link
Contributor Author

drupol commented Dec 2, 2021

Ok! I will amend.

Had to run away, but will fix this asap before lunch

@drupol drupol marked this pull request as ready for review December 2, 2021 10:19
@drupol
Copy link
Contributor Author

drupol commented Dec 2, 2021

Done, I updated the classes where the parameter name was not matching the interface. I did not modified the interfaces in order to avoid breaking stuff in some other external projects.

@greg0ire greg0ire added this to the 2.2.4 milestone Dec 2, 2021
@greg0ire greg0ire merged commit 0e29e07 into doctrine:2.2.x Dec 2, 2021
@greg0ire
Copy link
Member

greg0ire commented Dec 2, 2021

Thanks @drupol !

@drupol
Copy link
Contributor Author

drupol commented Dec 2, 2021

Merci :)

derrabus added a commit to derrabus/persistence that referenced this pull request Dec 19, 2021
* 2.2.x:
  Bump PHPStan and Psalm (doctrine#214)
  PHPStan 1.0.1
derrabus added a commit to derrabus/persistence that referenced this pull request Dec 19, 2021
* 2.3.x: (24 commits)
  Bump PHPStan and Psalm (doctrine#214)
  Update `ObjectRepository`, more type inference. (doctrine#213)
  PHPStan 1.0.1
  Reuse workflows from upstream
  Specify proxy type
  Undeprecate detach
  Fix class name
  Test on PHP 8.1
  Deprecate short namespace alias syntax
  Normalize continuous integration workflow
  Add explicit `@return` type next to `#[ReturnTypeWillChange]`
  Describe criteria as array with string keys
  Allow null as return value for type of field
  Mark doctrine/annotations dependency as optional
  Allow null as a return value for association's target (doctrine#192)
  Add .github, .doctrine-project.json and phpstan-baseline.neon to export ignore
  Reordered property type resolution to resolve typed properties before untyped ones
  Merge 2.2.x into 2.3.x (doctrine#191)
  Don't pass null to strpos() (doctrine#190)
  Add ReturnTypeWillChange to extended reflection classes
  ...
derrabus added a commit that referenced this pull request Dec 19, 2021
* 2.3.x: (24 commits)
  Bump PHPStan and Psalm (#214)
  Update `ObjectRepository`, more type inference. (#213)
  PHPStan 1.0.1
  Reuse workflows from upstream
  Specify proxy type
  Undeprecate detach
  Fix class name
  Test on PHP 8.1
  Deprecate short namespace alias syntax
  Normalize continuous integration workflow
  Add explicit `@return` type next to `#[ReturnTypeWillChange]`
  Describe criteria as array with string keys
  Allow null as return value for type of field
  Mark doctrine/annotations dependency as optional
  Allow null as a return value for association's target (#192)
  Add .github, .doctrine-project.json and phpstan-baseline.neon to export ignore
  Reordered property type resolution to resolve typed properties before untyped ones
  Merge 2.2.x into 2.3.x (#191)
  Don't pass null to strpos() (#190)
  Add ReturnTypeWillChange to extended reflection classes
  ...
derrabus added a commit to derrabus/persistence that referenced this pull request Dec 19, 2021
* 2.3.x: (24 commits)
  Bump PHPStan and Psalm (doctrine#214)
  Update `ObjectRepository`, more type inference. (doctrine#213)
  PHPStan 1.0.1
  Reuse workflows from upstream
  Specify proxy type
  Undeprecate detach
  Fix class name
  Test on PHP 8.1
  Deprecate short namespace alias syntax
  Normalize continuous integration workflow
  Add explicit `@return` type next to `#[ReturnTypeWillChange]`
  Describe criteria as array with string keys
  Allow null as return value for type of field
  Mark doctrine/annotations dependency as optional
  Allow null as a return value for association's target (doctrine#192)
  Add .github, .doctrine-project.json and phpstan-baseline.neon to export ignore
  Reordered property type resolution to resolve typed properties before untyped ones
  Merge 2.2.x into 2.3.x (doctrine#191)
  Don't pass null to strpos() (doctrine#190)
  Add ReturnTypeWillChange to extended reflection classes
  ...
derrabus added a commit to derrabus/persistence that referenced this pull request Dec 19, 2021
* 2.3.x: (24 commits)
  Bump PHPStan and Psalm (doctrine#214)
  Update `ObjectRepository`, more type inference. (doctrine#213)
  PHPStan 1.0.1
  Reuse workflows from upstream
  Specify proxy type
  Undeprecate detach
  Fix class name
  Test on PHP 8.1
  Deprecate short namespace alias syntax
  Normalize continuous integration workflow
  Add explicit `@return` type next to `#[ReturnTypeWillChange]`
  Describe criteria as array with string keys
  Allow null as return value for type of field
  Mark doctrine/annotations dependency as optional
  Allow null as a return value for association's target (doctrine#192)
  Add .github, .doctrine-project.json and phpstan-baseline.neon to export ignore
  Reordered property type resolution to resolve typed properties before untyped ones
  Merge 2.2.x into 2.3.x (doctrine#191)
  Don't pass null to strpos() (doctrine#190)
  Add ReturnTypeWillChange to extended reflection classes
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants