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

ISSUE-75: Added example on skipping settings.php generation. #77

Merged
merged 1 commit into from
Dec 5, 2024

Conversation

vishalkhode1
Copy link
Contributor

Motivation
Added an example which developers can follow to skip settings.php generation for certain use cases.

Proposed changes
Example file added showing on how to skip settings.php generation.

Alternatives considered
The Alternative thought be provide the similar way that BLT provides, but want to keep it simple and extensible, so ideal approach could be giving flexibility to developers, where they can create their own library/drush command and add their own checks to skip the settings.php generation logic.

Testing steps

  1. Include the example drush command in project.
  2. Set the environment variable i.e CI=true.
  3. Run the command drush drs:init:settings and you'll notice generation of settings.php is skipped..

Copy link
Contributor

@apathak18 apathak18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-11-29 at 6 20 39 PM Screenshot 2024-11-29 at 6 21 44 PM

@vishalkhode1 vishalkhode1 merged commit 5c010ff into develop Dec 5, 2024
4 checks passed
@vishalkhode1 vishalkhode1 deleted the ISSUE-75 branch December 5, 2024 13:09
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.

DRS is not compatible with BLT making migration not possible
2 participants