Skip to content

Ensure the order of database updates shown is consistent with how it would actually run (14.x) - #6520

Open
codebymikey wants to merge 2 commits into
drush-ops:14.xfrom
codebymikey:consistent-db-update-order-14.x
Open

Ensure the order of database updates shown is consistent with how it would actually run (14.x)#6520
codebymikey wants to merge 2 commits into
drush-ops:14.xfrom
codebymikey:consistent-db-update-order-14.x

Conversation

@codebymikey

Copy link
Copy Markdown
Contributor

14.x version of the #6376 PR.

@codebymikey codebymikey changed the title Ensure the order of database updates shown is consistent with how it would actually run Ensure the order of database updates shown is consistent with how it would actually run (14.x) Feb 23, 2026
@codebymikey
codebymikey force-pushed the consistent-db-update-order-14.x branch from 076ec6e to be09eea Compare February 23, 2026 12:12
@codebymikey
codebymikey force-pushed the consistent-db-update-order-14.x branch from be09eea to 8ef1056 Compare April 21, 2026 10:05
@weitzman

weitzman commented Jul 7, 2026

Copy link
Copy Markdown
Member

This looks good to me. At the same time, I'm hesitant to make any changes given that Drush is on its way out in favor of the new CLI in Drupal core. Any chance you are willing to code up an MR for updatedb in core? I would pair up to get that reviewed and RTBC.

@codebymikey

Copy link
Copy Markdown
Contributor Author

Any chance you are willing to code up an MR for updatedb in core?

Hmm yeah sure, I can take a crack at it since all the code is already here. What's the best way to do that? Is there an open issue with a list of functionality that needs to be implemented?

I'm hesitant to make any changes given that Drush is on its way out in favor of the new CLI in Drupal core.

I'm not sure about the timelines until Drupal core CLI will have feature-parity with Drush, but from having a quick look, it seems like there's still going to be a fairly long period where both will be used until dr encompasses drush feature-wise, so I struggle to see why a fix like this couldn't land until Drush is fully deprecated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants