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

Speed up many commands — remove checks for config management #24

Open
dalin- opened this issue Sep 19, 2023 · 3 comments
Open

Speed up many commands — remove checks for config management #24

dalin- opened this issue Sep 19, 2023 · 3 comments
Assignees

Comments

@dalin-
Copy link
Contributor

dalin- commented Sep 19, 2023

Do we or would we ever try to use these commands on a D8/9/10 site that doesn't use Configuration Management? I'm pretty confidently saying "no". So we can remove the checks for Config module, and Config Split (this check was only important because in Drupal 8.0 and 8.1 you needed two different drush commands for sites with/without it).

Removing this might save as much as 30s on:

  • pantheon-deploy
  • pantheon-multidev-*
Screenshot 2023-09-19 at 2 56 26 PM
@Yuvania
Copy link
Contributor

Yuvania commented Sep 21, 2023

@dalin- I can do it?

@Yuvania Yuvania self-assigned this Sep 21, 2023
@dalin-
Copy link
Contributor Author

dalin- commented Sep 21, 2023

@Yuvania
You're the maintainer, so you do whatever you like ;-)

@Yuvania
Copy link
Contributor

Yuvania commented Sep 21, 2023

You're the maintainer, so you do whatever you like ;-)

GREAT :D

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

No branches or pull requests

2 participants