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

Adjust config directory logic check on config import script #47

Open
duncancm9 opened this issue Dec 7, 2023 · 1 comment
Open

Adjust config directory logic check on config import script #47

duncancm9 opened this issue Dec 7, 2023 · 1 comment

Comments

@duncancm9
Copy link

The config directory can be in different paths and not necessarily in the path defined here:

https://github.com/fourkitchens/pots/blob/main/scripts/pantheon/drush-config-import#L20-L21

This can cause config to not be imported when desired.

@generalredneck
Copy link
Collaborator

The point of the logic is to make sure the drush cim command isn't run on Drupal 7 sites. I think I have some logic I've added recently to help with this. Use drush eval or drush status to get the drupal version. See for an code check using the VERSION constant. https://github.com/fourkitchens/evercurrent-v2/blob/main/web/modules/custom/evercurrent_server/src/DrushCommand/PhpEval/ProjectUpdates.php#L2-L23

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