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

update help for RunDqlCommand #10233

Merged
merged 2 commits into from
Nov 15, 2022
Merged

update help for RunDqlCommand #10233

merged 2 commits into from
Nov 15, 2022

Conversation

dmaicher
Copy link
Contributor

We are in the process of deprecating all commands on DoctrineBundle that are just extending the ORM command classes without adding anymore functionality.

We noticed that for the RunDqlCommand the help text on the DoctrineBundle command seems a bit more helpful.

So I'm proposing to update the help text here with what we have on DoctrineBundle.

See doctrine/DoctrineBundle#1581

@dmaicher dmaicher marked this pull request as ready for review November 15, 2022 16:08
@greg0ire greg0ire merged commit 958d0b6 into doctrine:2.14.x Nov 15, 2022
@dmaicher dmaicher deleted the update_help branch November 15, 2022 16:44
derrabus added a commit to derrabus/orm that referenced this pull request Nov 20, 2022
* 2.14.x:
  Remove fragile assertions (doctrine#10239)
  update help for RunDqlCommand (doctrine#10233)
  Make the code easier to statically analyse
  Widen parameter type
  Document property as non-nullable
derrabus added a commit to derrabus/orm that referenced this pull request Nov 20, 2022
* 2.14.x:
  Remove fragile assertions (doctrine#10239)
  update help for RunDqlCommand (doctrine#10233)
  Make the code easier to statically analyse
  Widen parameter type
  Document property as non-nullable
derrabus added a commit to derrabus/orm that referenced this pull request Nov 20, 2022
* 2.14.x:
  Remove fragile assertions (doctrine#10239)
  update help for RunDqlCommand (doctrine#10233)
  Make the code easier to statically analyse
  Widen parameter type
  Document property as non-nullable
derrabus added a commit to derrabus/orm that referenced this pull request Nov 20, 2022
* 2.14.x:
  Remove fragile assertions (doctrine#10239)
  update help for RunDqlCommand (doctrine#10233)
  Make the code easier to statically analyse
  Widen parameter type
  Document property as non-nullable
@derrabus derrabus added this to the 2.14.0 milestone Nov 21, 2022
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.

None yet

4 participants