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

Document stdClass structures used by CommitOrderCalculator #10315

Merged
merged 1 commit into from
Dec 19, 2022

Conversation

derrabus
Copy link
Member

This PR fixes some static analysis errors that popped up while I attempted to upgrade to Psalm 5.3. CommitOrderCalculator internally uses undocumented object structures which I've replaced with actual classes.

@derrabus derrabus added this to the 2.14.0 milestone Dec 18, 2022
@derrabus derrabus requested a review from greg0ire December 18, 2022 23:22
@derrabus derrabus force-pushed the improvement/co-vertex branch from afb4f16 to 658ac8a Compare December 19, 2022 09:48
@derrabus derrabus mentioned this pull request Dec 19, 2022
greg0ire
greg0ire previously approved these changes Dec 19, 2022
@derrabus derrabus force-pushed the improvement/co-vertex branch from 658ac8a to b318401 Compare December 19, 2022 14:36
@derrabus derrabus merged commit 82a4063 into doctrine:2.14.x Dec 19, 2022
@derrabus derrabus deleted the improvement/co-vertex branch December 19, 2022 14:43
derrabus added a commit to derrabus/orm that referenced this pull request Dec 19, 2022
* 2.14.x:
  Drop forceful loading of annotations (doctrine#10321)
  Document stdClass structures used by CommitOrderCalculator (doctrine#10315)
  Psalm 5.3.0 (doctrine#10317)
  PHPStan 1.9.4 (doctrine#10318)
  add apcu enabled check if apcu extension loaded (doctrine#10310) (doctrine#10311)
  Add TypedFieldMapper for automatic mapping of typed PHP fields to DBAL types (doctrine#10313)
derrabus added a commit to derrabus/orm that referenced this pull request Dec 19, 2022
* 2.14.x:
  Drop forceful loading of annotations (doctrine#10321)
  Document stdClass structures used by CommitOrderCalculator (doctrine#10315)
  Psalm 5.3.0 (doctrine#10317)
  PHPStan 1.9.4 (doctrine#10318)
  add apcu enabled check if apcu extension loaded (doctrine#10310) (doctrine#10311)
  Add TypedFieldMapper for automatic mapping of typed PHP fields to DBAL types (doctrine#10313)
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.

2 participants