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

Deprecate AbstractHydrator::hydrateRow() #9072

Merged
merged 1 commit into from
Oct 3, 2021

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Oct 3, 2021

While removing the iterate() methods, I noticed that the public method AbstractHydrator::hydrateRow() has become obsolete because we don't call it anymore. I'd like to remove it as well.

@derrabus derrabus added this to the 2.11.0 milestone Oct 3, 2021
@derrabus derrabus force-pushed the deprecate/hydrate-row branch 2 times, most recently from c95f870 to f26002a Compare October 3, 2021 21:23
@greg0ire greg0ire marked this pull request as draft October 3, 2021 21:36
@greg0ire
Copy link
Member

greg0ire commented Oct 3, 2021

Converting to draft for obvious reasons 😛

@derrabus derrabus force-pushed the deprecate/hydrate-row branch from f26002a to efebcb9 Compare October 3, 2021 21:38
@derrabus derrabus marked this pull request as ready for review October 3, 2021 21:38
@derrabus derrabus enabled auto-merge (squash) October 3, 2021 21:42
@derrabus derrabus merged commit ebdced6 into doctrine:2.11.x Oct 3, 2021
@derrabus derrabus deleted the deprecate/hydrate-row branch October 3, 2021 21:53
derrabus added a commit to derrabus/orm that referenced this pull request Oct 3, 2021
* 2.11.x:
  Deprecate AbstractHydrator::hydrateRow() (doctrine#9072)
  Reflect latest minor release in metadata
  Revert "Pin laminas/automatic-releases to 1.11.1"
  Revert "Try using docker image directly"
  Revert "Explicitly disallow workflows for tags"
  Revert "Use org admin token"
derrabus added a commit to derrabus/orm that referenced this pull request Oct 3, 2021
* 2.11.x:
  Deprecate AbstractHydrator::hydrateRow() (doctrine#9072)
  Reflect latest minor release in metadata
  Revert "Pin laminas/automatic-releases to 1.11.1"
  Revert "Try using docker image directly"
  Revert "Explicitly disallow workflows for tags"
  Revert "Use org admin token"

Signed-off-by: Alexander M. Turek <[email protected]>
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